-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
Test codegen of atomic compare-exchange with additional memory orderings #99409
Conversation
tmiasko
commented
Jul 18, 2022
- Add a test for atomic operations introduced in Simplify memory ordering intrinsics #97423 & Remove restrictions on compare-exchange memory ordering. #98383.
- Add a test for fallback code generation strategy used on LLVM 12 introduced in Work around llvm 12's memory ordering restrictions. #98385. Use a separate test case instead of a revision system since test will be gone once LLVM 12 is no longer supported.
(rust-highfive has picked a reviewer for you, use r? to override) |
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.
Thanks!
r=me with the allow(invalid_atomic_ordering)
lines removed.
* Add a test for atomic operations introduced in rust-lang#97423 & rust-lang#98383. * Add a test for fallback code generation strategy used on LLVM 12 introduced in rust-lang#98385. Use a separate test case instead of a revision system since test will be gone once LLVM 12 is no longer supported.
@bors r=m-ou-se |
⌛ Testing commit 8ebaf14 with merge 93af409f6fa48333e63390d60286ab68c2151389... |
💔 Test failed - checks-actions |
The job Click to see the possible cause of the failure (guessed by this bot)
|
@bors retry Could not resolve host: github.com |
☀️ Test successful - checks-actions |
Finished benchmarking commit (2a990f7): comparison url. Instruction countThis benchmark run did not return any relevant results for this metric. Max RSS (memory usage)This benchmark run did not return any relevant results for this metric. CyclesResults
If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf. @rustbot label: -perf-regression Footnotes |