Hi!
On IRC Richi mentioned some FAILs in gcc.target/x86_64 and in pr83126.c.
The following patch fixes the former ones (they need recent binutils to
be enabled), for pr83126.c because I didn't have graphite configured I've
just verified that the test compiles (didn't without the patch) and that
the gimple dump is identical with one from yesterday's gcc (as it was a
tree-parloops.cc ICE, I guess identical gimple is all we care about
and no need to verify it further).
Ok for trunk?
2023-12-01 Jakub Jelinek <ja...@redhat.com>
* gcc.target/x86_64/abi/avx512fp16/m512h/test_passing_m512.c
(fun_check_passing_m512_8_values, fun_check_passing_m512h_8_values):
Add missing void return type.
* gcc.target/x86_64/abi/avx512fp16/m256h/test_passing_m256.c
(fun_check_passing_m256_8_values, fun_check_passing_m256h_8_values):
Likewise.
* gcc.dg/graphite/pr83126.c (ew): Add missing casts to __INTPTR_TYPE__
and then to int *.
--- gcc/testsuite/gcc.target/x86_64/abi/avx512fp16/m512h/test_passing_m512.c.jj
2021-12-30 15:12:43.747143127 +0100
+++ gcc/testsuite/gcc.target/x86_64/abi/avx512fp16/m512h/test_passing_m512.c
2023-12-01 11:56:10.708574470 +0100
@@ -25,6 +25,7 @@ int failed = 0;
assert (memcmp (&X1, &X2, sizeof (T)) == 0); \
} while (0)
+void
fun_check_passing_m512_8_values (__m512 i0 ATTRIBUTE_UNUSED,
__m512 i1 ATTRIBUTE_UNUSED,
__m512 i2 ATTRIBUTE_UNUSED,
@@ -45,6 +46,7 @@ fun_check_passing_m512_8_values (__m512
compare (values.i7, i7, __m512);
}
+void
fun_check_passing_m512h_8_values (__m512h i0 ATTRIBUTE_UNUSED,
__m512h i1 ATTRIBUTE_UNUSED,
__m512h i2 ATTRIBUTE_UNUSED,
--- gcc/testsuite/gcc.target/x86_64/abi/avx512fp16/m256h/test_passing_m256.c.jj
2021-12-30 15:12:43.746143141 +0100
+++ gcc/testsuite/gcc.target/x86_64/abi/avx512fp16/m256h/test_passing_m256.c
2023-12-01 11:55:56.770772491 +0100
@@ -25,6 +25,7 @@ int failed = 0;
assert (memcmp (&X1, &X2, sizeof (T)) == 0); \
} while (0)
+void
fun_check_passing_m256_8_values (__m256 i0 ATTRIBUTE_UNUSED,
__m256 i1 ATTRIBUTE_UNUSED,
__m256 i2 ATTRIBUTE_UNUSED,
@@ -45,6 +46,7 @@ fun_check_passing_m256_8_values (__m256
compare (values.i7, i7, __m256);
}
+void
fun_check_passing_m256h_8_values (__m256h i0 ATTRIBUTE_UNUSED,
__m256h i1 ATTRIBUTE_UNUSED,
__m256h i2 ATTRIBUTE_UNUSED,
--- gcc/testsuite/gcc.dg/graphite/pr83126.c.jj 2020-01-12 11:54:37.438397944
+0100
+++ gcc/testsuite/gcc.dg/graphite/pr83126.c 2023-12-01 12:20:42.045695863
+0100
@@ -12,7 +12,7 @@ ew (unsigned short int c9, int stuff)
int *fd = &stuff;
*fd = c9;
- fd = *fd;
+ fd = (int *) (__INTPTR_TYPE__) *fd;
if (*fd != 0)
for (*by = 0; *by < 2; ++*by)
c9 *= e1;
Jakub
