-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
Re-enable ThinLTO for rustc on x86_64-apple-darwin
#105845
Conversation
This reverts commit ddb6fe2.
@bors try |
⌛ Trying commit a270a348bc99a57e25b39e58db466f928f03439b with merge 16aba2156f3ed344308e345c000744d0476fd9ec... |
r=me on the re-enable presuming try builds look good |
☀️ Try build successful - checks-actions |
a270a34
to
ae68e17
Compare
This is looking good, the I've dropped the temporary CI builder hack. For bisection and perf we can avoid rolling it up, but it shouldn't really matter. @bors r=Mark-Simulacrum rollup=never |
⌛ Testing commit ae68e17 with merge f6f6328ec504cb9ab8ef1234282ff206d65f3657... |
💥 Test timed out |
@bors retry - x86_64-gnu-llvm-13-stage1 was unresponsive while compiling rustbuild right at the beginning |
This comment was marked as outdated.
This comment was marked as outdated.
☀️ Test successful - checks-actions |
Finished benchmarking commit (1072337): 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)ResultsThis 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.
CyclesThis benchmark run did not return any relevant results for this metric. |
ThinLTO was disabled on x64 mac in #105646 because of the #105637 regression.
It was later discovered that the issue was present on other targets as well, as the mac revert was already landing. The linux/win reverts, however, did not land before the root cause was identified.
#105800 fixed the underlying issue in
-Zdylib-lto
handling, and the x64 msvc and linux targets are now fixed, ICEs are using the correctrustc_driver
panic hook.This PR re-enables ThinLTO on mac for improved perf now that the issue should be fixed everywhere.