Remove big-endian swizzles from vreinterpret#1919
Remove big-endian swizzles from vreinterpret#1919folkertdev merged 1 commit intorust-lang:masterfrom
vreinterpret#1919Conversation
|
r? @folkertdev rustbot has assigned @folkertdev. Use |
|
I'm also really confused why the intrinsic-test tool doesn't catch this. Printing out the values you can see that the intrinsic does indeed get tested and it matches C. At the same time this PR clearly changes their behaviour but still passes the test tool |
|
I've confirmed locally that the rust Just confirming with the Arm folks: the implementation in this PR is the correct one, right? |
Looking through how it is implemented in |
Fixes: #1904
This is the correct behavior according to the Intrinsics Guide, which specifies that these should compile to
NOP. Also, GCC and Clang matches this behavior.I am a bit surprised that this didn't get caught in the AArch64_BE
intrinsic_testrun!