diff mbox

[AArch64] Removed unused get_lane and dup_lane builtins.

Message ID 53DBADAB.6010705@arm.com
State New
Headers show

Commit Message

Alan Lawrence Aug. 1, 2014, 3:09 p.m. UTC 
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.

----------

Comments

Marcus Shawcroft Aug. 1, 2014, 3:45 p.m. UTC | #1 
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
diff mbox

Patch

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)