* [Patch, Aarch64] Testsuite fix to fix one of the regressions in PR 87433, gcc.dg/zero_bits_compound-1.c
@ 2018-09-26 14:59 Steve Ellcey
2018-09-26 18:11 ` Jeff Law
0 siblings, 1 reply; 2+ messages in thread
From: Steve Ellcey @ 2018-09-26 14:59 UTC (permalink / raw)
To: gcc-patches
PR rtl-optimization/85160 which allowed combine to convert
two instructions into two different instructions if they had a lower cost
caused a couple of regressions on aarch64. This patch fixes one of them.
After the above patch, the gcc.dg/zero_bits_compound-1.c test on
aarch64 generates an and rtl instruction where it did not before.
A comparision of the code generated before and after the 85160 patch
shows that GCC is still generating the same number of instructions
but there are fewer dependencies between some of the instructions
so the new code would be better on systems with multiple functional
units and no worse on systems with a single functional unit.
Since the test is just verifying that an 'and' rtl instruction is
not generated and that is now happening on aarch64 but the new code
is as good as or better than the old code this test doesn't seem
to make sense for aarch64 anymore and I would like to just not
run it on aarch64.
Tested on aarch64. OK for checkin?
Steve Ellcey
sellcey@cavium.com
2018-09-26 Steve Ellcey <sellcey@cavium.com>
PR testsuite/87433
* gcc.dg/zero_bits_compound-1.c: Do not run on aarch64*-*-*.
diff --git a/gcc/testsuite/gcc.dg/zero_bits_compound-1.c b/gcc/testsuite/gcc.dg/zero_bits_compound-1.c
index 63b8ac1..e715949 100644
--- a/gcc/testsuite/gcc.dg/zero_bits_compound-1.c
+++ b/gcc/testsuite/gcc.dg/zero_bits_compound-1.c
@@ -4,7 +4,7 @@
/* Note: This test requires that char, int and long have different sizes and the
target has a way to do 32 -> 64 bit zero extension other than AND. */
-/* { dg-do compile { target i?86-*-* x86_64-*-* s390*-*-* aarch64*-*-* } } */
+/* { dg-do compile { target i?86-*-* x86_64-*-* s390*-*-* } } */
/* { dg-require-effective-target lp64 } */
/* { dg-options "-O3 -dP" } */
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [Patch, Aarch64] Testsuite fix to fix one of the regressions in PR 87433, gcc.dg/zero_bits_compound-1.c
2018-09-26 14:59 [Patch, Aarch64] Testsuite fix to fix one of the regressions in PR 87433, gcc.dg/zero_bits_compound-1.c Steve Ellcey
@ 2018-09-26 18:11 ` Jeff Law
0 siblings, 0 replies; 2+ messages in thread
From: Jeff Law @ 2018-09-26 18:11 UTC (permalink / raw)
To: sellcey, gcc-patches
On 9/26/18 8:58 AM, Steve Ellcey wrote:
> PR rtl-optimization/85160 which allowed combine to convert
> two instructions into two different instructions if they had a lower cost
> caused a couple of regressions on aarch64.  This patch fixes one of them.
>
> After the above patch, the gcc.dg/zero_bits_compound-1.c test onÂ
> aarch64 generates an and rtl instruction where it did not before.
> A comparision of the code generated before and after the 85160 patch
> shows that GCC is still generating the same number of instructions
> but there are fewer dependencies between some of the instructions
> so the new code would be better on systems with multiple functional
> units and no worse on systems with a single functional unit.
>
> Since the test is just verifying that an 'and' rtl instruction is
> not generated and that is now happening on aarch64 but the new code
> is as good as or better than the old code this test doesn't seem
> to make sense for aarch64 anymore and I would like to just not
> run it on aarch64.
>
> Tested on aarch64.  OK for checkin?
>
> Steve Ellcey
> sellcey@cavium.com
>
>
> 2018-09-26  Steve Ellcey  <sellcey@cavium.com>
>
> PR testsuite/87433
> * gcc.dg/zero_bits_compound-1.c: Do not run on aarch64*-*-*.
OK.
jeff
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2018-09-26 18:10 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-09-26 14:59 [Patch, Aarch64] Testsuite fix to fix one of the regressions in PR 87433, gcc.dg/zero_bits_compound-1.c Steve Ellcey
2018-09-26 18:11 ` Jeff Law
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).