-
Notifications
You must be signed in to change notification settings - Fork 13k
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
Delete the cfg(not(parallel))
serial compiler
#132282
Conversation
cc @davidtwco, @compiler-errors, @TaKO8Ki This PR modifies If appropriate, please update |
fae56ab
to
2e9696f
Compare
cc @davidtwco, @compiler-errors, @TaKO8Ki This PR modifies If appropriate, please update |
This comment has been minimized.
This comment has been minimized.
2e9696f
to
9c4a078
Compare
9c4a078
to
2c453de
Compare
This PR modifies If appropriate, please update |
☔ The latest upstream changes (presumably #132513) made this pull request unmergeable. Please resolve the merge conflicts. |
2c453de
to
09d2f62
Compare
@bors p=1 many changes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I trust you cleaned-up the cfgs correctly.
Some cases of missed formatting for macros there and there, if you don't mind fixing them.
Does this need some kind of heads-up to the team?
r=me if doesn't.
From the discussion I think we can land this. @Noratrieb Could you help doing the format work as @cjgillot suggested? |
Yes, I will do it, but it may take me a few days. Feel free to push the fix to my branch and approve it if you want to. |
Since it's inception a long time ago, the parallel compiler and its cfgs have been a maintenance burden. This was a necessary evil the allow iteration while not degrading performance because of synchronization overhead. But this time is over. Thanks to the amazing work by the parallel working group (and the dyn sync crimes), the parallel compiler has now been fast enough to be shipped by default in nightly for quite a while now. Stable and beta have still been on the serial compiler, because they can't use `-Zthreads` anyways. But this is quite suboptimal: - the maintenance burden still sucks - we're not testing the serial compiler in nightly Because of these reasons, it's time to end it. The serial compiler has served us well in the years since it was split from the parallel one, but it's over now. Let the knight slay one head of the two-headed dragon!
09d2f62
to
505b8e1
Compare
@bors r=cjgillot rollup=never |
☀️ Test successful - checks-actions |
Finished benchmarking commit (6503543): comparison URL. Overall result: no relevant changes - no action needed@rustbot label: -perf-regression Instruction countThis benchmark run did not return any relevant results for this metric. Max RSS (memory usage)Results (primary -1.0%)This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
CyclesResults (secondary 2.0%)This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 787.726s -> 784.558s (-0.40%) |
Since it's inception a long time ago, the parallel compiler and its cfgs have been a maintenance burden. This was a necessary evil the allow iteration while not degrading performance because of synchronization overhead.
But this time is over. Thanks to the amazing work by the parallel working group (and the dyn sync crimes), the parallel compiler has now been fast enough to be shipped by default in nightly for quite a while now.
Stable and beta have still been on the serial compiler, because they can't use
-Zthreads
anyways.But this is quite suboptimal:
Because of these reasons, it's time to end it. The serial compiler has served us well in the years since it was split from the parallel one, but it's over now.
Let the knight slay one head of the two-headed dragon!
#113349
Note that the default is still 1 thread, as more than 1 thread is still fairly broken.
cc @onur-ozkan to see if i did the bootstrap field removal correctly, @SparrowLii on the sync parts