Really, we were already 99% of the way there; just needed a couple minor fixes that affected 64-bit-only builtins. Based on D61717.
Note that the change to builtin_str changes the type of a few __builtin_neon_* intrinsics that had the "wrong" type.
Paths
| Differential D68683
ARM] Fix arm_neon.h with -flax-vector-conversions=none ClosedPublic Authored by efriedma on Oct 8 2019, 8:29 PM.
Details Summary Really, we were already 99% of the way there; just needed a couple minor fixes that affected 64-bit-only builtins. Based on D61717. Note that the change to builtin_str changes the type of a few __builtin_neon_* intrinsics that had the "wrong" type.
Diff Detail
Event TimelineThis revision is now accepted and ready to land.Oct 9 2019, 12:15 AM Comment Actions Thank you! We also have a hack in the ARM and AArch64 target info to set the default for lax vector conversions back to "all" when NEON is enabled that can now be removed. Closed by commit rG4c4df441860c: [ARM] Fix arm_neon.h with -flax-vector-conversions=none (authored by efriedma). · Explain WhyOct 9 2019, 10:58 AM This revision was automatically updated to reflect the committed changes. Comment Actions
Never mind, looks like I didn't get round to landing D67678 yet. =)
Revision Contents
Diff 224105 clang/test/CodeGen/aarch64-neon-2velem.c
clang/test/CodeGen/aarch64-neon-intrinsics.c
clang/test/CodeGen/aarch64-neon-scalar-copy.c
clang/test/CodeGen/aarch64-neon-scalar-x-indexed-elem.c
clang/test/CodeGen/aarch64-neon-vget.c
clang/test/CodeGen/aarch64-poly64.c
clang/test/CodeGen/aarch64-v8.2a-neon-intrinsics.c
clang/test/CodeGen/arm64-lanes.c
clang/test/CodeGen/arm64_vcopy.c
clang/test/CodeGen/arm_neon_intrinsics.c
clang/test/Headers/arm-neon-header.c
clang/utils/TableGen/NeonEmitter.cpp
|