> -----Original Message-----
> From: Christophe Lyon <christophe.l...@linaro.org>
> Sent: Thursday, November 16, 2023 3:26 PM
> To: gcc-patches@gcc.gnu.org; Richard Sandiford
> <richard.sandif...@arm.com>; Richard Earnshaw
> <richard.earns...@arm.com>; Kyrylo Tkachov <kyrylo.tkac...@arm.com>
> Cc: Christophe Lyon <christophe.l...@linaro.org>
> Subject: [PATCH 5/6] arm: [MVE intrinsics] fix vst1 tests
>
> vst1q intrinsics return void, so we should not do 'return vst1q_f16 (base,
> value);'
>
> This was OK so far, but will trigger an error/warning with the new
> implementation of these intrinsics.
>
Whoops!
Ok (could have gone in as obvious IMO).
Thanks,
Kyrill
> This patch just removes the 'return' keyword.
>
> 2023-11-16 Christophe Lyon <christophe.l...@linaro.org>
>
> gcc/testsuite/
> * gcc.target/arm/mve/intrinsics/vst1q_f16.c: Remove 'return'.
> * gcc.target/arm/mve/intrinsics/vst1q_f32.c: Likewise.
> * gcc.target/arm/mve/intrinsics/vst1q_s16.c: Likewise.
> * gcc.target/arm/mve/intrinsics/vst1q_s32.c: Likewise.
> * gcc.target/arm/mve/intrinsics/vst1q_s8.c: Likewise.
> * gcc.target/arm/mve/intrinsics/vst1q_u16.c: Likewise.
> * gcc.target/arm/mve/intrinsics/vst1q_u32.c: Likewise.
> * gcc.target/arm/mve/intrinsics/vst1q_u8.c: Likewise.
> ---
> gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_f16.c | 4 ++--
> gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_f32.c | 4 ++--
> gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_s16.c | 4 ++--
> gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_s32.c | 4 ++--
> gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_s8.c | 4 ++--
> gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_u16.c | 4 ++--
> gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_u32.c | 4 ++--
> gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_u8.c | 4 ++--
> 8 files changed, 16 insertions(+), 16 deletions(-)
>
> diff --git a/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_f16.c
> b/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_f16.c
> index 1fa02f00f53..e4b40604d54 100644
> --- a/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_f16.c
> +++ b/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_f16.c
> @@ -18,7 +18,7 @@ extern "C" {
> void
> foo (float16_t *base, float16x8_t value)
> {
> - return vst1q_f16 (base, value);
> + vst1q_f16 (base, value);
> }
>
>
> @@ -31,7 +31,7 @@ foo (float16_t *base, float16x8_t value)
> void
> foo1 (float16_t *base, float16x8_t value)
> {
> - return vst1q (base, value);
> + vst1q (base, value);
> }
>
> #ifdef __cplusplus
> diff --git a/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_f32.c
> b/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_f32.c
> index 67cc3ae3b47..8f42323c603 100644
> --- a/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_f32.c
> +++ b/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_f32.c
> @@ -18,7 +18,7 @@ extern "C" {
> void
> foo (float32_t *base, float32x4_t value)
> {
> - return vst1q_f32 (base, value);
> + vst1q_f32 (base, value);
> }
>
>
> @@ -31,7 +31,7 @@ foo (float32_t *base, float32x4_t value)
> void
> foo1 (float32_t *base, float32x4_t value)
> {
> - return vst1q (base, value);
> + vst1q (base, value);
> }
>
> #ifdef __cplusplus
> diff --git a/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_s16.c
> b/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_s16.c
> index 052959b2083..891ac4155d9 100644
> --- a/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_s16.c
> +++ b/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_s16.c
> @@ -18,7 +18,7 @@ extern "C" {
> void
> foo (int16_t *base, int16x8_t value)
> {
> - return vst1q_s16 (base, value);
> + vst1q_s16 (base, value);
> }
>
>
> @@ -31,7 +31,7 @@ foo (int16_t *base, int16x8_t value)
> void
> foo1 (int16_t *base, int16x8_t value)
> {
> - return vst1q (base, value);
> + vst1q (base, value);
> }
>
> #ifdef __cplusplus
> diff --git a/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_s32.c
> b/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_s32.c
> index 444ad07f4ef..a28d1eb98db 100644
> --- a/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_s32.c
> +++ b/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_s32.c
> @@ -18,7 +18,7 @@ extern "C" {
> void
> foo (int32_t *base, int32x4_t value)
> {
> - return vst1q_s32 (base, value);
> + vst1q_s32 (base, value);
> }
>
>
> @@ -31,7 +31,7 @@ foo (int32_t *base, int32x4_t value)
> void
> foo1 (int32_t *base, int32x4_t value)
> {
> - return vst1q (base, value);
> + vst1q (base, value);
> }
>
> #ifdef __cplusplus
> diff --git a/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_s8.c
> b/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_s8.c
> index 684ff0aca5b..81c141a63e0 100644
> --- a/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_s8.c
> +++ b/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_s8.c
> @@ -18,7 +18,7 @@ extern "C" {
> void
> foo (int8_t *base, int8x16_t value)
> {
> - return vst1q_s8 (base, value);
> + vst1q_s8 (base, value);
> }
>
>
> @@ -31,7 +31,7 @@ foo (int8_t *base, int8x16_t value)
> void
> foo1 (int8_t *base, int8x16_t value)
> {
> - return vst1q (base, value);
> + vst1q (base, value);
> }
>
> #ifdef __cplusplus
> diff --git a/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_u16.c
> b/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_u16.c
> index 1fea2de1e76..b8ce7fbe6ee 100644
> --- a/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_u16.c
> +++ b/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_u16.c
> @@ -18,7 +18,7 @@ extern "C" {
> void
> foo (uint16_t *base, uint16x8_t value)
> {
> - return vst1q_u16 (base, value);
> + vst1q_u16 (base, value);
> }
>
>
> @@ -31,7 +31,7 @@ foo (uint16_t *base, uint16x8_t value)
> void
> foo1 (uint16_t *base, uint16x8_t value)
> {
> - return vst1q (base, value);
> + vst1q (base, value);
> }
>
> #ifdef __cplusplus
> diff --git a/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_u32.c
> b/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_u32.c
> index 64c43c59d47..1dbb55538a9 100644
> --- a/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_u32.c
> +++ b/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_u32.c
> @@ -18,7 +18,7 @@ extern "C" {
> void
> foo (uint32_t *base, uint32x4_t value)
> {
> - return vst1q_u32 (base, value);
> + vst1q_u32 (base, value);
> }
>
>
> @@ -31,7 +31,7 @@ foo (uint32_t *base, uint32x4_t value)
> void
> foo1 (uint32_t *base, uint32x4_t value)
> {
> - return vst1q (base, value);
> + vst1q (base, value);
> }
>
> #ifdef __cplusplus
> diff --git a/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_u8.c
> b/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_u8.c
> index 5517611bba6..ab22be81647 100644
> --- a/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_u8.c
> +++ b/gcc/testsuite/gcc.target/arm/mve/intrinsics/vst1q_u8.c
> @@ -18,7 +18,7 @@ extern "C" {
> void
> foo (uint8_t *base, uint8x16_t value)
> {
> - return vst1q_u8 (base, value);
> + vst1q_u8 (base, value);
> }
>
>
> @@ -31,7 +31,7 @@ foo (uint8_t *base, uint8x16_t value)
> void
> foo1 (uint8_t *base, uint8x16_t value)
> {
> - return vst1q (base, value);
> + vst1q (base, value);
> }
>
> #ifdef __cplusplus
> --
> 2.34.1
