add lib tests for vec::IntoIter alignment issues #106112
Conversation
rustbot
added
S-waiting-on-review
|
Hey! It looks like you've submitted a new PR for the library teams! If this PR contains changes to any Examples of
|
| impl Drop for AlignedZstWithDrop { | ||
| fn drop(&mut self) { | ||
| let addr = self as *mut _ as usize; | ||
| assert!(hint::black_box(addr) % mem::align_of::<u64>() == 0); |
There was a problem hiding this comment.
Seems reasonable enough. Good thing we didn't call it hint::bench_black_box :p
There was a problem hiding this comment.
Yeah I guess it is useful for some opportunistic UB-detection tests as well. ;)
|
@bors r+ rollup |
bors
added
S-waiting-on-bors
|
☀️ Test successful - checks-actions |
|
Finished benchmarking commit (d9ee0f4): 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. |
This adds non-Miri tests for the issue fixed in #106084
r? @thomcc