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" } } */ +