[PATCH][AArch64] Fix types for vqdmlals_lane_s32 and vqdmlsls_lane_s32 intrinsics

Richard Earnshaw rearnsha@arm.com
Tue Aug 5 10:42:00 GMT 2014


On 04/08/14 17:49, Kyrill Tkachov wrote:
> Hi all,
> 
> Now that uint64x1_t and uint64_t are not interchangeable these 
> intrinsics have an incorrect type. The reason is that these intrinsics 
> should have had the scalar type for some of their arguments and results 
> according to the NEON intrinsics spec.
> 
> This patch fixes that and updates the relevant tests.
> 
> Tested on aarch64-none-elf and bootstrapped on aarch64-linux.
> 
> Ok for trunk?
> 
> 2014-08-04  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>
> 
>      * config/aarch64/arm_neon.h (vqdmlals_lane_s32): Use scalar types
>      rather than singleton vectors.
>      (vqdmlsls_lane_s32): Likewise.
> 
> 2014-08-04  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>
> 
>      * gcc.target/aarch64/scalar_intrinsics.c (test_vqdmlals_lane_s32):
>      Fix types.
>      (test_vqdmlsls_lane_s32): Likewise.
>      * gcc.target/aarch64/simd/vqdmlals_lane_s32.c: Likewise.
>      * gcc.target/aarch64/simd/vqdmlsls_lane_s32.c: Likewise.
> 
> 

OK.

R.




More information about the Gcc-patches mailing list