Jonathan Wright via Gcc-patches <[email protected]> writes:
> The pointer parameter to load a vector of signed values should itself
> be a signed type. This patch fixes two instances of this unsigned-
> signed implicit conversion in arm_neon.h.
>
> Tested relevant intrinsics with -Wpointer-sign and warnings no longer
> present.
>
> Ok for master?
>
> Thanks,
> Jonathan
>
> ---
>
> gcc/ChangeLog:
>
> 2021-10-14 Jonathan Wright <[email protected]>
>
> * config/aarch64/arm_neon.h (vld1_s8_x3): Use signed type for
> pointer parameter.
> (vld1_s32_x3): Likewise.
OK, thanks.
Richard
> diff --git a/gcc/config/aarch64/arm_neon.h b/gcc/config/aarch64/arm_neon.h
> index
> 2d5bf34b698a88ed934c522cc9f14f125c604a39..24068f8d7da5b123360a479f66b53fe2d4eea28e
> 100644
> --- a/gcc/config/aarch64/arm_neon.h
> +++ b/gcc/config/aarch64/arm_neon.h
> @@ -16198,7 +16198,7 @@ vld1_u8_x3 (const uint8_t *__a)
>
> __extension__ extern __inline int8x8x3_t
> __attribute__ ((__always_inline__, __gnu_inline__, __artificial__))
> -vld1_s8_x3 (const uint8_t *__a)
> +vld1_s8_x3 (const int8_t *__a)
> {
> int8x8x3_t __i;
> __builtin_aarch64_simd_ci __o;
> @@ -16250,7 +16250,7 @@ vld1_u32_x3 (const uint32_t *__a)
>
> __extension__ extern __inline int32x2x3_t
> __attribute__ ((__always_inline__, __gnu_inline__, __artificial__))
> -vld1_s32_x3 (const uint32_t *__a)
> +vld1_s32_x3 (const int32_t *__a)
> {
> int32x2x3_t __i;
> __builtin_aarch64_simd_ci __o;
