loongarch: Avoid constant folding in tests to ensure SIMD coverage#2075
loongarch: Avoid constant folding in tests to ensure SIMD coverage#2075heiher wants to merge 1 commit intorust-lang:mainfrom
Conversation
|
r? @sayantn rustbot has assigned @sayantn. Use Why was this reviewer chosen?The reviewer was selected based on:
|
heiher
force-pushed
the
loong64-tests-avoid-const-folding
branch
from
f1fa465 to
4bdc281
Compare
|
This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed. Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers. |
| -111, -98, 47, -106, -82, -72, -70, 0, 110, -61, -20, 36, 41, -103, 42, 95, 15, -11, | ||
| -25, -5, 40, -63, 56, -39, 43, 127, 86, 75, -48, -32, 72, 69, |
There was a problem hiding this comment.
that's weird, why did this formatting change?? The changes only added black_box!
There was a problem hiding this comment.
Did the new rustfmt version change the default settings? All I did was regenerate and run rustfmt.
There was a problem hiding this comment.
This was caused by a change in rustfmt's max-width. I've reformatted the code, and the diff should be clean now.
Use `black_box` on SIMD intrinsic inputs to prevent the compiler from constant folding SIMD operations, ensuring the corresponding SIMD instructions are actually emitted and covered by tests.
heiher
force-pushed
the
loong64-tests-avoid-const-folding
branch
from
4bdc281 to
ab81338
Compare
3 participants
Use
black_boxon SIMD intrinsic inputs to prevent the compiler from constant folding SIMD operations, ensuring the corresponding SIMD instructions are actually emitted and covered by tests.