* Re: [PATCH] testsuite: Tweak some further tests for modern C changes
2023-12-01 11:27 [PATCH] testsuite: Tweak some further tests for modern C changes Jakub Jelinek
@ 2023-12-01 11:24 ` Richard Biener
2023-12-01 11:37 ` Florian Weimer
1 sibling, 0 replies; 3+ messages in thread
From: Richard Biener @ 2023-12-01 11:24 UTC (permalink / raw)
To: Jakub Jelinek; +Cc: Florian Weimer, Jeff Law, gcc-patches
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 <jakub@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 <rguenther@suse.de>
SUSE Software Solutions Germany GmbH,
Frankenstrasse 146, 90461 Nuernberg, Germany;
GF: Ivo Totev, Andrew McDonald, Werner Knoblich; (HRB 36809, AG Nuernberg)
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] testsuite: Tweak some further tests for modern C changes
@ 2023-12-01 11:27 Jakub Jelinek
2023-12-01 11:24 ` Richard Biener
2023-12-01 11:37 ` Florian Weimer
0 siblings, 2 replies; 3+ messages in thread
From: Jakub Jelinek @ 2023-12-01 11:27 UTC (permalink / raw)
To: Richard Biener, Florian Weimer, Jeff Law; +Cc: gcc-patches
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 <jakub@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
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] testsuite: Tweak some further tests for modern C changes
2023-12-01 11:27 [PATCH] testsuite: Tweak some further tests for modern C changes Jakub Jelinek
2023-12-01 11:24 ` Richard Biener
@ 2023-12-01 11:37 ` Florian Weimer
1 sibling, 0 replies; 3+ messages in thread
From: Florian Weimer @ 2023-12-01 11:37 UTC (permalink / raw)
To: Jakub Jelinek; +Cc: Richard Biener, Jeff Law, gcc-patches
* Jakub Jelinek:
> 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 <jakub@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 *.
Looks fine. Sorry, I totally forgot to upgrade binutils and install
isl.
Thanks,
Florian
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-12-01 11:37 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-12-01 11:27 [PATCH] testsuite: Tweak some further tests for modern C changes Jakub Jelinek
2023-12-01 11:24 ` Richard Biener
2023-12-01 11:37 ` Florian Weimer
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).