Skip to content

Commit ae3d387

Browse files
committed
Auto merge of #42500 - hsivonen:neon, r=japaric
Change `llvm.neon.*` to `llvm.arm.neon.*` in the mapping for platform intrinsics This avoids linker errors when using platform intrinsics on 32-bit ARM with NEON. Fixes hsivonen/simd#10. I don't have an explanation why the old code might have worked when the `simd` crate was written, since the new LLVM intrinsic naming seems to have existed as early as 2012. Maybe LLVM accepted two naming schemes for a few years and dropped the `llvm.neon.*` version recently? I don't know. Also, I don't know how to write a unit test for this.
2 parents c799b37 + 0fb8414 commit ae3d387

File tree

2 files changed

+391
-391
lines changed

2 files changed

+391
-391
lines changed

src/etc/platform-intrinsics/arm.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
{
22
"platform": "arm_v",
33
"intrinsic_prefix": "",
4-
"llvm_prefix": "llvm.neon.v",
4+
"llvm_prefix": "llvm.arm.neon.v",
55
"number_info": {
66
"signed": {
77
"kind": "s",

0 commit comments

Comments
 (0)