[Bug testsuite/91458] FAIL: g++.dg/tree-ssa/pr19807.C -std=gnu++98 scan-tree-dump-times optimized "\\\\[\\\\(void .\\\\) \\\\+ 8B\\\\]" 3

2019-08-20 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91458

--- Comment #7 from Martin Sebor  ---
Author: msebor
Date: Wed Aug 21 02:18:41 2019
New Revision: 274764

URL: https://gcc.gnu.org/viewcvs?rev=274764=gcc=rev
Log:
PR testsuite/91458

gcc/testsuite/ChangeLog:
* g++.dg/tree-ssa/ssa-dse-1.C: Use the same search pattern
unconditionally (correcting r272199, PR middle-end/90676).
* gcc.dg/tree-prof/stringop-2.c: Same.

Modified:
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/g++.dg/tree-ssa/ssa-dse-1.C
trunk/gcc/testsuite/gcc.dg/tree-prof/stringop-2.c

[Bug testsuite/91458] FAIL: g++.dg/tree-ssa/pr19807.C -std=gnu++98 scan-tree-dump-times optimized "\\\\[\\\\(void .\\\\) \\\\+ 8B\\\\]" 3

2019-08-17 Thread dave.anglin at bell dot net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91458

--- Comment #6 from dave.anglin at bell dot net ---
On 2019-08-15 10:47 a.m., danglin at gcc dot gnu.org wrote:
> Similar fail:
> FAIL: g++.dg/tree-ssa/ssa-dse-1.C   scan-tree-dump-times dse1
> "MEM[(struct FixBuf *)& + [0-9]+B] = {}" 1
>
> Here we have:
>   MEM  [(struct FixBuf *)& + 28B] = {};
This test fails for the same reason as pr19807.C.  I noticed this morning that
gcc.dg/tree-prof/stringop-2.c
also fails for same reason:
FAIL: gcc.dg/tree-prof/stringop-2.c scan-tree-dump optimized "MEM[(void
.)] = 168430090"

Here we have the following in optimized tree dump:
  MEM  [(void *)] = 168430090;

[Bug testsuite/91458] FAIL: g++.dg/tree-ssa/pr19807.C -std=gnu++98 scan-tree-dump-times optimized "\\\\[\\\\(void .\\\\) \\\\+ 8B\\\\]" 3

2019-08-16 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91458

Martin Sebor  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #5 from Martin Sebor  ---
I've made the change in r274574.

[Bug testsuite/91458] FAIL: g++.dg/tree-ssa/pr19807.C -std=gnu++98 scan-tree-dump-times optimized "\\\\[\\\\(void .\\\\) \\\\+ 8B\\\\]" 3

2019-08-16 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91458

--- Comment #4 from Martin Sebor  ---
Author: msebor
Date: Fri Aug 16 15:47:25 2019
New Revision: 274574

URL: https://gcc.gnu.org/viewcvs?rev=274574=gcc=rev
Log:
PR testsuite/91458

gcc/testsuite/ChangeLog:

* g++.dg/tree-ssa/pr19807.C: Use the same search pattern
unconditionally (correcting r272199, PR middle-end/90676).


Modified:
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/g++.dg/tree-ssa/pr19807.C

[Bug testsuite/91458] FAIL: g++.dg/tree-ssa/pr19807.C -std=gnu++98 scan-tree-dump-times optimized "\\\\[\\\\(void .\\\\) \\\\+ 8B\\\\]" 3

2019-08-16 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91458

Martin Sebor  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|unassigned at gcc dot gnu.org  |msebor at gcc dot 
gnu.org

--- Comment #3 from Martin Sebor  ---
Let me fix it.

[Bug testsuite/91458] FAIL: g++.dg/tree-ssa/pr19807.C -std=gnu++98 scan-tree-dump-times optimized "\\\\[\\\\(void .\\\\) \\\\+ 8B\\\\]" 3

2019-08-16 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91458

Martin Sebor  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2019-08-16
 Ever confirmed|0   |1

--- Comment #2 from Martin Sebor  ---
I think it's just a mistake in the pattern:

/* { dg-final { scan-tree-dump-times "\\\[\\\(void .\\\) \\\+ 8B\\\]" 3
"optimized" { target { ! store_merge } } } }
   { dg-final { scan-tree-dump-times "  \\\[\\\(void .\\\) \\\+
8B\\\]" 3 "optimized" { target { store_merge } } } } */

No store merging takes place here so the test should use the second pattern
unconditionally.

[Bug testsuite/91458] FAIL: g++.dg/tree-ssa/pr19807.C -std=gnu++98 scan-tree-dump-times optimized "\\\\[\\\\(void .\\\\) \\\\+ 8B\\\\]" 3

2019-08-16 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91458

Richard Biener  changed:

   What|Removed |Added

 CC||msebor at gcc dot gnu.org
  Component|tree-optimization   |testsuite

--- Comment #1 from Richard Biener  ---
Martin changed the dumping