i386/testsuite: Correct res_ref2 array size for avx512bw-vpmov{, us}wb-2.c

Message ID 20250916064644.1743001-1-haochen.jiang@intel.com
State Committed
Commit 39c7b08d4ea33254c74ac0d2d7c2420ab1cb7f98
Headers
Series i386/testsuite: Correct res_ref2 array size for avx512bw-vpmov{, us}wb-2.c |

Commit Message

Jiang, Haochen Sept. 16, 2025, 6:46 a.m. UTC
  Both of the tests under 128 bit are raising:

warning: writing 16 bytes into a region of size 8 [-Wstringop-overflow=]

when compiling, leading to a test fail. The warning is caused by the
incorrect array size for res_ref2. The wrong size caused the overflow.

Correct them in this patch to fix the test fail.

Ok for trunk?

Thx,
Haochen

gcc/testsuite/ChangeLog:

	* gcc.target/i386/avx512bw-vpmovuswb-2.c: Correct res_ref2
	array size.
	* gcc.target/i386/avx512bw-vpmovwb-2.c: Ditto.
---
 gcc/testsuite/gcc.target/i386/avx512bw-vpmovuswb-2.c | 2 +-
 gcc/testsuite/gcc.target/i386/avx512bw-vpmovwb-2.c   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
  

Comments

liuhongt Sept. 16, 2025, 6:47 a.m. UTC | #1
> -----Original Message-----
> From: Jiang, Haochen <haochen.jiang@intel.com>
> Sent: Tuesday, September 16, 2025 2:47 PM
> To: gcc-patches@gcc.gnu.org
> Cc: Liu, Hongtao <hongtao.liu@intel.com>; ubizjak@gmail.com
> Subject: [PATCH] i386/testsuite: Correct res_ref2 array size for avx512bw-
> vpmov{,us}wb-2.c
> 
> Both of the tests under 128 bit are raising:
> 
> warning: writing 16 bytes into a region of size 8 [-Wstringop-overflow=]
> 
> when compiling, leading to a test fail. The warning is caused by the incorrect
> array size for res_ref2. The wrong size caused the overflow.
> 
> Correct them in this patch to fix the test fail.
> 
> Ok for trunk?
Ok.
> 
> Thx,
> Haochen
> 
> gcc/testsuite/ChangeLog:
> 
> 	* gcc.target/i386/avx512bw-vpmovuswb-2.c: Correct res_ref2
> 	array size.
> 	* gcc.target/i386/avx512bw-vpmovwb-2.c: Ditto.
> ---
>  gcc/testsuite/gcc.target/i386/avx512bw-vpmovuswb-2.c | 2 +-
>  gcc/testsuite/gcc.target/i386/avx512bw-vpmovwb-2.c   | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/gcc/testsuite/gcc.target/i386/avx512bw-vpmovuswb-2.c
> b/gcc/testsuite/gcc.target/i386/avx512bw-vpmovuswb-2.c
> index d5198f39eb6..2311dfea34f 100644
> --- a/gcc/testsuite/gcc.target/i386/avx512bw-vpmovuswb-2.c
> +++ b/gcc/testsuite/gcc.target/i386/avx512bw-vpmovuswb-2.c
> @@ -27,7 +27,7 @@ TEST (void)
>    UNION_TYPE (AVX512F_LEN, i_w) src;
>    MASK_TYPE mask = MASK_VALUE;
>    unsigned char res_ref[32];
> -  unsigned char res_ref2[SIZE];
> +  unsigned char res_ref2[SIZE_HALF];
> 
>    for (i = 0; i < SIZE; i++)
>      {
> diff --git a/gcc/testsuite/gcc.target/i386/avx512bw-vpmovwb-2.c
> b/gcc/testsuite/gcc.target/i386/avx512bw-vpmovwb-2.c
> index 6b0f86f57d8..ea91181284f 100644
> --- a/gcc/testsuite/gcc.target/i386/avx512bw-vpmovwb-2.c
> +++ b/gcc/testsuite/gcc.target/i386/avx512bw-vpmovwb-2.c
> @@ -28,7 +28,7 @@ TEST (void)
>    UNION_TYPE (AVX512F_LEN, i_w) src;
>    MASK_TYPE mask = MASK_VALUE;
>    char res_ref[32];
> -  char res_ref2[SIZE];
> +  char res_ref2[SIZE_HALF];
> 
>    sign = -1;
>    for (i = 0; i < SIZE; i++)
> --
> 2.31.1
  

Patch

diff --git a/gcc/testsuite/gcc.target/i386/avx512bw-vpmovuswb-2.c b/gcc/testsuite/gcc.target/i386/avx512bw-vpmovuswb-2.c
index d5198f39eb6..2311dfea34f 100644
--- a/gcc/testsuite/gcc.target/i386/avx512bw-vpmovuswb-2.c
+++ b/gcc/testsuite/gcc.target/i386/avx512bw-vpmovuswb-2.c
@@ -27,7 +27,7 @@  TEST (void)
   UNION_TYPE (AVX512F_LEN, i_w) src;
   MASK_TYPE mask = MASK_VALUE;
   unsigned char res_ref[32];
-  unsigned char res_ref2[SIZE];
+  unsigned char res_ref2[SIZE_HALF];
 
   for (i = 0; i < SIZE; i++)
     {
diff --git a/gcc/testsuite/gcc.target/i386/avx512bw-vpmovwb-2.c b/gcc/testsuite/gcc.target/i386/avx512bw-vpmovwb-2.c
index 6b0f86f57d8..ea91181284f 100644
--- a/gcc/testsuite/gcc.target/i386/avx512bw-vpmovwb-2.c
+++ b/gcc/testsuite/gcc.target/i386/avx512bw-vpmovwb-2.c
@@ -28,7 +28,7 @@  TEST (void)
   UNION_TYPE (AVX512F_LEN, i_w) src;
   MASK_TYPE mask = MASK_VALUE;
   char res_ref[32];
-  char res_ref2[SIZE];
+  char res_ref2[SIZE_HALF];
 
   sign = -1;
   for (i = 0; i < SIZE; i++)