Message ID | 53DBADAB.6010705@arm.com |
---|---|
State | New |
Headers | show |
On 1 August 2014 16:09, Alan Lawrence <alan.lawrence@arm.com> wrote: > None of the variants of __builtin_aarch64_get_lane or > __builtin_aarch64_dup_lane are used in arm_neon.h (vdup_lane uses vget_lane > and vdup_n, vget_lane uses be_checked_get_lane to do an endianness swap, > vdup_n uses gcc vector extension code). So remove them. > > Regression tested on aarch64-none-elf. > > gcc/ChangeLog: > > * gcc/config/aarch64/aarch64-simd-builtins.def > (dup_lane, get_lane): Delete. OK /Marcus
Index: gcc/config/aarch64/aarch64-simd-builtins.def =================================================================== --- gcc/config/aarch64/aarch64-simd-builtins.def (revision 213452) +++ gcc/config/aarch64/aarch64-simd-builtins.def (working copy) @@ -47,8 +47,6 @@ VAR1 (UNOP, addp, 0, di) BUILTIN_VDQ_BHSI (UNOP, clz, 2) - BUILTIN_VALL (GETLANE, get_lane, 0) - VAR1 (GETLANE, get_lane, 0, di) BUILTIN_VALL (GETLANE, be_checked_get_lane, 0) VAR1 (REINTERP_SS, reinterpretdi, 0, v1df) @@ -74,7 +72,6 @@ VAR1 (REINTERP_PS, reinterpretv2si, 0, v1df) VAR1 (REINTERP_PS, reinterpretv2sf, 0, v1df) - BUILTIN_VDQ_I (BINOP, dup_lane, 0) /* Implemented by aarch64_<sur>q<r>shl<mode>. */ BUILTIN_VSDQ_I (BINOP, sqshl, 0) BUILTIN_VSDQ_I (BINOP_UUS, uqshl, 0)