On Fri, 1 Dec 2023, Jakub Jelinek wrote: > 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?
OK. Richard. > 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 > > -- Richard Biener <rguent...@suse.de> SUSE Software Solutions Germany GmbH, Frankenstrasse 146, 90461 Nuernberg, Germany; GF: Ivo Totev, Andrew McDonald, Werner Knoblich; (HRB 36809, AG Nuernberg)
