On targets where the callee may make a copy of incoming aggregates DSE
would trigger in both functions -- prior to inlining of course.

This patch tightens the test to look for DSE triggering in the spot
where we really wanted to check for it.  It's not strictly a regression
fix, but given it's a testsuite only change it seems appropriate.

Verified it fixes the failure on the hppa targets (prior to the ABI
change) as well as that it still passes on x86_64.

Installed on the trunk,

Jeff
commit 0a82247b5bb50c2fb62e334bc20c35a1654c10ca
Author: law <law@138bc75d-0d04-0410-961f-82ee72b054a4>
Date:   Thu Jan 18 04:05:27 2018 +0000

            PR testsuite/83883
            * gcc.dg/tree-ssa/ssa-dse-26.c: Tighten expected output.
    
    git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@256833 
138bc75d-0d04-0410-961f-82ee72b054a4

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index f710c158848..492c650911e 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2018-01-17  Jeff Law  <l...@redhat.com>
+
+       PR testsuite/83883
+       * gcc.dg/tree-ssa/ssa-dse-26.c: Tighten expected output.
+
 2018-01-17  Bill Schmidt  <wschm...@linux.vnet.ibm.com>
 
        * gcc.target/powerpc/safe-indirect-jump-1.c: Remove endian
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 a5638b58247..8e0a24a6c2c 100644
--- a/gcc/testsuite/gcc.dg/tree-ssa/ssa-dse-26.c
+++ b/gcc/testsuite/gcc.dg/tree-ssa/ssa-dse-26.c
@@ -29,4 +29,6 @@ constraint_equal (struct constraint a, struct constraint b)
     && constraint_expr_equal (a.rhs, b.rhs);
 }
 
-/* { dg-final { scan-tree-dump-times "Deleted dead store" 2 "dse1" } } */
+/* { dg-final { scan-tree-dump-times "Deleted dead store: x = " 1 "dse1" } } */
+/* { dg-final { scan-tree-dump-times "Deleted dead store: y = " 1 "dse1" } } */
+

Reply via email to