Minor testsuite fix for cr16 and xstormy16 targets

Message ID 64182845-0084-3d2d-61ef-463eaf46b4ac@gmail.com
State New
Headers show
Series
  • Minor testsuite fix for cr16 and xstormy16 targets
Related show

Commit Message

Bill Schmidt via Gcc-patches May 3, 2021, 2:55 p.m.
Richi's recent work twiddles dse1's actions for these targets. This 
patch just updates the expected output.  I'll keep an eye on other 
targets as their results trickle in.


Committed to the trunk,


Jeff
commit 4d8a13cad842e7822fa29d0aa93799033a0f7847
Author: Jeff Law <jlaw@tachyum.com>
Date:   Mon May 3 07:50:22 2021 -0700

    Fix test results on cr16 and xstormy16
    
    gcc/testsuite
            * gcc.dg/tree-ssa/ssa-dse-26.c: Update expected output for
            cr16 and xstormy16 targets

Comments

Bill Schmidt via Gcc-patches May 4, 2021, 8:33 a.m. | #1
On Mon, May 3, 2021 at 6:08 PM Jeff Law via Gcc-patches
<gcc-patches@gcc.gnu.org> wrote:
>

>

> Richi's recent work twiddles dse1's actions for these targets. This

> patch just updates the expected output.  I'll keep an eye on other

> targets as their results trickle in.

>

>

> Committed to the trunk,


Thanks - the same case apparently also happens for
x86_64 with -m32, I'll adjust for that.

Richard.

>

> Jeff

>
Bill Schmidt via Gcc-patches May 4, 2021, 2:53 p.m. | #2
On 5/4/2021 2:33 AM, Richard Biener wrote:
> On Mon, May 3, 2021 at 6:08 PM Jeff Law via Gcc-patches

> <gcc-patches@gcc.gnu.org> wrote:

>>

>> Richi's recent work twiddles dse1's actions for these targets. This

>> patch just updates the expected output.  I'll keep an eye on other

>> targets as their results trickle in.

>>

>>

>> Committed to the trunk,

> Thanks - the same case apparently also happens for

> x86_64 with -m32, I'll adjust for that.


Thanks.  I've got some more on the embedded side to commit/push.


Jeff

Patch

diff --git a/gcc/testsuite/gcc.dg/tree-ssa/ssa-dse-26.c b/gcc/testsuite/gcc.dg/tree-ssa/ssa-dse-26.c
index 271e666f265..1321fec308e 100644
--- a/gcc/testsuite/gcc.dg/tree-ssa/ssa-dse-26.c
+++ b/gcc/testsuite/gcc.dg/tree-ssa/ssa-dse-26.c
@@ -31,8 +31,8 @@  constraint_equal (struct constraint a, struct constraint b)
 }
 
 /* Most targets should be using this test.  */
-/* { dg-final { scan-tree-dump-times "Deleted dead store: x = " 1 "dse1" { target { ! { tic6x-*-* mmix-knuth-mmixware } } } } } */
-/* { dg-final { scan-tree-dump-times "Deleted dead store: y = " 1 "dse1" { target { ! { tic6x-*-* mmix-knuth-mmixware } } } } } */
+/* { dg-final { scan-tree-dump-times "Deleted dead store: x = " 1 "dse1" { target { ! { tic6x-*-* mmix-knuth-mmixware cr16*-*-* xstormy16-*-*-* } } } } } */
+/* { dg-final { scan-tree-dump-times "Deleted dead store: y = " 1 "dse1" { target { ! { tic6x-*-* mmix-knuth-mmixware cr16*-*-* xstormy16-*-*-* } } } } } */
 
 /* The c6x port generates significantly different gimple which
    changes the SRA and DSE decisions.   Verify we remove all
@@ -42,3 +42,6 @@  constraint_equal (struct constraint a, struct constraint b)
 /* { dg-final { scan-tree-dump-times "Deleted dead store: x::. = " 1 "dse1" { target mmix-knuth-mmixware } } } */
 /* { dg-final { scan-tree-dump-times "Deleted dead store: y::. = " 1 "dse1" { target mmix-knuth-mmixware } } } */
 
+/* And more special cases
+/* { dg-final { scan-tree-dump-times "Deleted dead store: x = " 2 "dse1" { target cr16*-*-* xstormy16-*-*-*} } } */
+/* { dg-final { scan-tree-dump-times "Deleted dead store: y = " 2 "dse1" { target cr16*-*-* xstormy16-*-*-*} } } */