Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Default rust.new-symbol-mangling to true universally #133998

Closed
wants to merge 1 commit into from

Conversation

jieyouxu
Copy link
Member

@jieyouxu jieyouxu commented Dec 7, 2024

FIXME: this needs to preserve mixed behavior for dist profile because v0 is not stable cc #60705 grrr

Instead of using legacy scheme for library but new scheme for compiler when the option is not set.

r? @ghost

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) labels Dec 7, 2024
Instead of using legacy scheme for library but new scheme for compiler
when the option is not set.
@jieyouxu jieyouxu force-pushed the consistent-mangling branch from 04c8536 to 6f449f3 Compare December 7, 2024 03:13
@jieyouxu
Copy link
Member Author

jieyouxu commented Dec 7, 2024

I don't think this is worth changing before we can make the full switch.

@jieyouxu jieyouxu closed this Dec 7, 2024
@jieyouxu jieyouxu deleted the consistent-mangling branch December 7, 2024 04:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants