-
Notifications
You must be signed in to change notification settings - Fork 13.1k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Add a regression test for argument copies with DestinationPropagation #106660
Conversation
e93cc8d
to
1801611
Compare
@bors r+ rollup=never codegen test |
📌 Commit 1801611b314a4205ff89580ab7a4f05dceb929af has been approved by It is now in the queue for this repository. |
⌛ Testing commit 1801611b314a4205ff89580ab7a4f05dceb929af with merge a4bdc47796082ecd6160777c17fcca83c424a64c... |
💔 Test failed - checks-actions |
This comment has been minimized.
This comment has been minimized.
1801611
to
4234757
Compare
The alignment of u64 is 8 on 64-bit systems but 4 on 32-bit, so the codegen test failed on a 32-bit target. I switched |
4234757
to
aa5e6c2
Compare
@bors r+ |
📌 Commit aa5e6c2a7b3edec0fb9534b45164fc2a7a62e5c5 has been approved by It is now in the queue for this repository. |
🌲 The tree is currently closed for pull requests below priority 999. This pull request will be tested once the tree is reopened. |
@bors delegate=saethlin |
✌️ @saethlin can now approve this pull request |
aa5e6c2
to
13eec69
Compare
@bors r=tmiasko rollup=never codegen test |
☀️ Test successful - checks-actions |
Finished benchmarking commit (1e4f900): 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.
CyclesResultsThis 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.
|
This example, as a codegen test: #105813 (comment)
r? @tmiasko