https://gcc.gnu.org/g:efb1d2e2368e60da3c691ee3cb510ee690d1fa2a
commit r15-5665-gefb1d2e2368e60da3c691ee3cb510ee690d1fa2a
Author: Haochen Jiang <haochen.ji...@intel.com>
Date: Fri Nov 22 15:57:47 2024 +0800
i386/testsuite: Correct AVX10.2 FP8 test mask usage
Under FP8, we should not use AVX512F_LEN_HALF to get the mask size since
it will get 16 instead of 8 and drop into wrong if condition. Correct
the usage for vcvtneph2[b,h]f8[,s] runtime test.
gcc/testsuite/ChangeLog:
* gcc.target/i386/avx10_2-512-vcvtneph2bf8-2.c: Correct 128bit
mask usage.
* gcc.target/i386/avx10_2-512-vcvtneph2bf8s-2.c: Ditto.
* gcc.target/i386/avx10_2-512-vcvtneph2hf8-2.c: Ditto.
* gcc.target/i386/avx10_2-512-vcvtneph2hf8s-2.c: Ditto.
Diff:
---
.../gcc.target/i386/avx10_2-512-vcvtneph2bf8-2.c | 25 +++++++++++++---------
.../gcc.target/i386/avx10_2-512-vcvtneph2bf8s-2.c | 25 +++++++++++++---------
.../gcc.target/i386/avx10_2-512-vcvtneph2hf8-2.c | 23 ++++++++++++--------
.../gcc.target/i386/avx10_2-512-vcvtneph2hf8s-2.c | 23 ++++++++++++--------
4 files changed, 58 insertions(+), 38 deletions(-)
diff --git a/gcc/testsuite/gcc.target/i386/avx10_2-512-vcvtneph2bf8-2.c
b/gcc/testsuite/gcc.target/i386/avx10_2-512-vcvtneph2bf8-2.c
index d5ba911334c8..96ca7e80c4d5 100644
--- a/gcc/testsuite/gcc.target/i386/avx10_2-512-vcvtneph2bf8-2.c
+++ b/gcc/testsuite/gcc.target/i386/avx10_2-512-vcvtneph2bf8-2.c
@@ -11,8 +11,8 @@
#include "avx10-helper.h"
#include "fp8-helper.h"
-#define SIZE_SRC (AVX512F_LEN / 16)
-#define SIZE (AVX512F_LEN_HALF / 8)
+#define SIZE (AVX512F_LEN / 16)
+#define SIZE_DST (AVX512F_LEN_HALF / 8)
#include "avx512f-mask-type.h"
void
@@ -23,14 +23,14 @@ CALC (unsigned char *r, _Float16 *s)
hf8_bf8 = 1;
saturate = 0;
- for (i = 0; i < SIZE; i++)
+ for (i = 0; i < SIZE_DST; i++)
{
r[i] = 0;
- if (i < SIZE_SRC)
- {
- Float16Union usrc = {.f16 = s[i]};
- r[i] = convert_fp16_to_fp8(usrc.f16, 0, hf8_bf8, saturate);
- }
+ if (i < SIZE)
+ {
+ Float16Union usrc = {.f16 = s[i]};
+ r[i] = convert_fp16_to_fp8(usrc.f16, 0, hf8_bf8, saturate);
+ }
}
}
@@ -41,17 +41,22 @@ TEST (void)
UNION_TYPE (AVX512F_LEN_HALF, i_b) res1, res2, res3;
UNION_TYPE (AVX512F_LEN, h) src;
MASK_TYPE mask = MASK_VALUE;
- unsigned char res_ref[SIZE];
+ unsigned char res_ref[SIZE_DST];
sign = 1;
- for (i = 0; i < SIZE_SRC; i++)
+ for (i = 0; i < SIZE; i++)
{
src.a[i] = (_Float16)(sign * (2.5 * (1 << (i % 3))));
sign = -sign;
}
+#if AVX512F_LEN > 128
+ for (i = 0; i < SIZE_DST; i++)
+ res2.a[i] = DEFAULT_VALUE;
+#else
for (i = 0; i < SIZE; i++)
res2.a[i] = DEFAULT_VALUE;
+#endif
CALC(res_ref, src.a);
diff --git a/gcc/testsuite/gcc.target/i386/avx10_2-512-vcvtneph2bf8s-2.c
b/gcc/testsuite/gcc.target/i386/avx10_2-512-vcvtneph2bf8s-2.c
index 49e170aa4288..c458f1ebb779 100644
--- a/gcc/testsuite/gcc.target/i386/avx10_2-512-vcvtneph2bf8s-2.c
+++ b/gcc/testsuite/gcc.target/i386/avx10_2-512-vcvtneph2bf8s-2.c
@@ -11,8 +11,8 @@
#include "avx10-helper.h"
#include "fp8-helper.h"
-#define SIZE_SRC (AVX512F_LEN / 16)
-#define SIZE (AVX512F_LEN_HALF / 8)
+#define SIZE (AVX512F_LEN / 16)
+#define SIZE_DST (AVX512F_LEN_HALF / 8)
#include "avx512f-mask-type.h"
void
@@ -23,14 +23,14 @@ CALC (unsigned char *r, _Float16 *s)
hf8_bf8 = 1;
saturate = 1;
- for (i = 0; i < SIZE; i++)
+ for (i = 0; i < SIZE_DST; i++)
{
r[i] = 0;
- if (i < SIZE_SRC)
- {
- Float16Union usrc = {.f16 = s[i]};
- r[i] = convert_fp16_to_fp8(usrc.f16, 0, hf8_bf8, saturate);
- }
+ if (i < SIZE)
+ {
+ Float16Union usrc = {.f16 = s[i]};
+ r[i] = convert_fp16_to_fp8(usrc.f16, 0, hf8_bf8, saturate);
+ }
}
}
@@ -41,17 +41,22 @@ TEST (void)
UNION_TYPE (AVX512F_LEN_HALF, i_b) res1, res2, res3;
UNION_TYPE (AVX512F_LEN, h) src;
MASK_TYPE mask = MASK_VALUE;
- unsigned char res_ref[SIZE];
+ unsigned char res_ref[SIZE_DST];
sign = 1;
- for (i = 0; i < SIZE_SRC; i++)
+ for (i = 0; i < SIZE; i++)
{
src.a[i] = (_Float16)(sign * (2.5 * (1 << (i % 3))));
sign = -sign;
}
+#if AVX512F_LEN > 128
+ for (i = 0; i < SIZE_DST; i++)
+ res2.a[i] = DEFAULT_VALUE;
+#else
for (i = 0; i < SIZE; i++)
res2.a[i] = DEFAULT_VALUE;
+#endif
CALC(res_ref, src.a);
diff --git a/gcc/testsuite/gcc.target/i386/avx10_2-512-vcvtneph2hf8-2.c
b/gcc/testsuite/gcc.target/i386/avx10_2-512-vcvtneph2hf8-2.c
index f481b72cc71f..cb9cdbb89c1d 100644
--- a/gcc/testsuite/gcc.target/i386/avx10_2-512-vcvtneph2hf8-2.c
+++ b/gcc/testsuite/gcc.target/i386/avx10_2-512-vcvtneph2hf8-2.c
@@ -11,8 +11,8 @@
#include "avx10-helper.h"
#include "fp8-helper.h"
-#define SIZE_SRC (AVX512F_LEN / 16)
-#define SIZE (AVX512F_LEN_HALF / 8)
+#define SIZE (AVX512F_LEN / 16)
+#define SIZE_DST (AVX512F_LEN_HALF / 8)
#include "avx512f-mask-type.h"
void
@@ -23,14 +23,14 @@ CALC (unsigned char *r, _Float16 *s)
hf8_bf8 = 0;
saturate = 0;
- for (i = 0; i < SIZE; i++)
+ for (i = 0; i < SIZE_DST; i++)
{
r[i] = 0;
- if (i < SIZE_SRC)
- {
- Float16Union usrc = {.f16 = s[i]};
- r[i] = convert_fp16_to_fp8(usrc.f16, 0, hf8_bf8, saturate);
- }
+ if (i < SIZE)
+ {
+ Float16Union usrc = {.f16 = s[i]};
+ r[i] = convert_fp16_to_fp8(usrc.f16, 0, hf8_bf8, saturate);
+ }
}
}
@@ -44,14 +44,19 @@ TEST (void)
unsigned char res_ref[SIZE];
sign = 1;
- for (i = 0; i < SIZE_SRC; i++)
+ for (i = 0; i < SIZE; i++)
{
src.a[i] = (_Float16)(sign * (2.5 * (1 << (i % 3))));
sign = -sign;
}
+#if AVX512F_LEN > 128
+ for (i = 0; i < SIZE_DST; i++)
+ res2.a[i] = DEFAULT_VALUE;
+#else
for (i = 0; i < SIZE; i++)
res2.a[i] = DEFAULT_VALUE;
+#endif
CALC(res_ref, src.a);
diff --git a/gcc/testsuite/gcc.target/i386/avx10_2-512-vcvtneph2hf8s-2.c
b/gcc/testsuite/gcc.target/i386/avx10_2-512-vcvtneph2hf8s-2.c
index 4782b038f3b1..4827af4df533 100644
--- a/gcc/testsuite/gcc.target/i386/avx10_2-512-vcvtneph2hf8s-2.c
+++ b/gcc/testsuite/gcc.target/i386/avx10_2-512-vcvtneph2hf8s-2.c
@@ -11,8 +11,8 @@
#include "avx10-helper.h"
#include "fp8-helper.h"
-#define SIZE_SRC (AVX512F_LEN / 16)
-#define SIZE (AVX512F_LEN_HALF / 8)
+#define SIZE (AVX512F_LEN / 16)
+#define SIZE_DST (AVX512F_LEN_HALF / 8)
#include "avx512f-mask-type.h"
void
@@ -23,14 +23,14 @@ CALC (unsigned char *r, _Float16 *s)
hf8_bf8 = 0;
saturate = 1;
- for (i = 0; i < SIZE; i++)
+ for (i = 0; i < SIZE_DST; i++)
{
r[i] = 0;
- if (i < SIZE_SRC)
- {
- Float16Union usrc = {.f16 = s[i]};
- r[i] = convert_fp16_to_fp8(usrc.f16, 0, hf8_bf8, saturate);
- }
+ if (i < SIZE)
+ {
+ Float16Union usrc = {.f16 = s[i]};
+ r[i] = convert_fp16_to_fp8(usrc.f16, 0, hf8_bf8, saturate);
+ }
}
}
@@ -44,14 +44,19 @@ TEST (void)
unsigned char res_ref[SIZE];
sign = 1;
- for (i = 0; i < SIZE_SRC; i++)
+ for (i = 0; i < SIZE; i++)
{
src.a[i] = (_Float16)(sign * (2.5 * (1 << (i % 3))));
sign = -sign;
}
+#if AVX512F_LEN > 128
+ for (i = 0; i < SIZE_DST; i++)
+ res2.a[i] = DEFAULT_VALUE;
+#else
for (i = 0; i < SIZE; i++)
res2.a[i] = DEFAULT_VALUE;
+#endif
CALC(res_ref, src.a);
