The ACLE spec which describes these intrinsics hasn't been published yet, but this is based on the final draft which will be published soon, and these have already been implemented by GCC.
Details
Diff Detail
- Repository
- rL LLVM
Event Timeline
I think this looks OK.
include/clang/Basic/arm_neon.td | ||
---|---|---|
1587 | Nit: I think it's obvious, but perhaps a comment here, which would be consistent with similar remarks in this file, that this this is for both A32 and A64 (and the one below A64 only)? | |
include/clang/Basic/arm_neon_incl.td | ||
256 | Yes, more modifiers! ;-) | |
test/CodeGen/aarch64-neon-dot-product.c | ||
6 | Same nit as in of the other patches: not sure if we should spell out Armv8.2-A or if v8.2-A is actually ok. |
test/CodeGen/arm-neon-dot-product.c | ||
---|---|---|
1 | I think proper target is cortex-a55 or cortex-a75. |
test/CodeGen/arm-neon-dot-product.c | ||
---|---|---|
1 | This is working as intended, I think: target-feature overrides target-cpu. |
test/CodeGen/arm-neon-dot-product.c | ||
---|---|---|
1 | dotprod is ARMv8.2's addition feature not ARMv8. Cortex-a57 only supports ARMv8 which couldn't have dotprod feature. Am I missing something? |
test/CodeGen/arm-neon-dot-product.c | ||
---|---|---|
1 | Oh, sorry, misunderstood the question; yes, this might not be the most clear "CHECK" line. |
test/CodeGen/arm-neon-dot-product.c | ||
---|---|---|
1 | I've changed this to cortex-a75 for clarity. |
Nit: I think it's obvious, but perhaps a comment here, which would be consistent with similar remarks in this file, that this this is for both A32 and A64 (and the one below A64 only)?