[Bug middle-end/90676] default GIMPLE dumps lack information
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90676 --- Comment #11 from Martin Sebor --- Author: msebor Date: Wed Aug 21 02:18:41 2019 New Revision: 274764 URL: https://gcc.gnu.org/viewcvs?rev=274764&root=gcc&view=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 middle-end/90676] default GIMPLE dumps lack information
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90676 --- Comment #10 from Martin Sebor --- Author: msebor Date: Fri Aug 16 15:47:25 2019 New Revision: 274574 URL: https://gcc.gnu.org/viewcvs?rev=274574&root=gcc&view=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 middle-end/90676] default GIMPLE dumps lack information
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90676 --- Comment #9 from Martin Sebor --- Author: msebor Date: Wed Jun 12 20:43:27 2019 New Revision: 272218 URL: https://gcc.gnu.org/viewcvs?rev=272218&root=gcc&view=rev Log: Tweak tests to avoid fallout from MEM_REF changes on targets that don't enable store merging but achieve the same effect by other means (DSE). gcc/testsuite?ChangeLog: PR middle-end/90676 * gcc.dg/tree-ssa/pr30375.c: Adjust and simplify expected test output some more. * gcc.dg/tree-ssa/slsr-27.c: Ditto. * gcc.dg/tree-ssa/slsr-28.c: Ditto. * gcc.dg/tree-ssa/slsr-29.c: Ditto. * gcc.dg/tree-ssa/ssa-dse-24.c: Ditto. Modified: trunk/gcc/testsuite/ChangeLog trunk/gcc/testsuite/gcc.dg/tree-ssa/pr30375.c trunk/gcc/testsuite/gcc.dg/tree-ssa/slsr-27.c trunk/gcc/testsuite/gcc.dg/tree-ssa/slsr-28.c trunk/gcc/testsuite/gcc.dg/tree-ssa/slsr-29.c trunk/gcc/testsuite/gcc.dg/tree-ssa/ssa-dse-24.c
[Bug middle-end/90676] default GIMPLE dumps lack information
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90676 Martin Sebor changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|--- |FIXED Target Milestone|--- |10.0 --- Comment #8 from Martin Sebor --- Adjusted patch committed in r272199.
[Bug middle-end/90676] default GIMPLE dumps lack information
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90676 --- Comment #7 from Martin Sebor --- Author: msebor Date: Wed Jun 12 17:05:35 2019 New Revision: 272199 URL: https://gcc.gnu.org/viewcvs?rev=272199&root=gcc&view=rev Log: PR middle-end/90676 - default GIMPLE dumps lack information gcc/ChangeLog: PR middle-end/90676 * tree-pretty-print.c (dump_mem_ref): New function. Include MEM_REF type in output when different size than operand. (dump_generic_node): Move code to dump_mem_ref and call it. gcc/testsuite/ChangeLog: PR middle-end/90676 * gcc.dg/tree-ssa/dump-6.c: New test. * g++.dg/tree-ssa/pr19807.C: Adjust expected output. * g++.dg/tree-ssa/ssa-dse-1.C: Same. * gcc.dg/store_merging_5.c: Same. * gcc.dg/tree-prof/stringop-2.c: Same. * gcc.dg/tree-ssa/pr30375.c: Same. * gcc.dg/tree-ssa/slsr-27.c: Same. * gcc.dg/tree-ssa/slsr-28.c: Same. * gcc.dg/tree-ssa/slsr-29.c: Same. * gcc.dg/tree-ssa/ssa-dse-24.c: Same. Added: trunk/gcc/testsuite/gcc.dg/tree-ssa/dump-6.c Modified: trunk/gcc/ChangeLog trunk/gcc/testsuite/ChangeLog trunk/gcc/testsuite/g++.dg/tree-ssa/pr19807.C trunk/gcc/testsuite/g++.dg/tree-ssa/ssa-dse-1.C trunk/gcc/testsuite/gcc.dg/store_merging_5.c trunk/gcc/testsuite/gcc.dg/tree-prof/stringop-2.c trunk/gcc/testsuite/gcc.dg/tree-ssa/pr30375.c trunk/gcc/testsuite/gcc.dg/tree-ssa/slsr-27.c trunk/gcc/testsuite/gcc.dg/tree-ssa/slsr-28.c trunk/gcc/testsuite/gcc.dg/tree-ssa/slsr-29.c trunk/gcc/testsuite/gcc.dg/tree-ssa/ssa-dse-24.c trunk/gcc/tree-pretty-print.c
[Bug middle-end/90676] default GIMPLE dumps lack information
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90676 Martin Sebor changed: What|Removed |Added Keywords||patch Status|NEW |ASSIGNED Assignee|unassigned at gcc dot gnu.org |msebor at gcc dot gnu.org --- Comment #6 from Martin Sebor --- Dropping the access type is like printing 'memset(a, 0)' and dropping the size argument. I think the simplest solution for MEM_REF is to mention the type. It may not include all the detail we might be interested in but it does provide the most essential information. Patch: https://gcc.gnu.org/ml/gcc-patches/2019-06/msg00011.html (PR 87052 and PR 90694 are a couple other examples that we've fixed in the past to get more meaningful/reliable dumps.)
[Bug middle-end/90676] default GIMPLE dumps lack information
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90676 Richard Biener changed: What|Removed |Added Summary|[9 Regression] ambiguous|default GIMPLE dumps lack |GIMPLE after store merging |information --- Comment #5 from Richard Biener --- The bugreport is of course bougs - there's no ambiguity in GIMPLE just the (default) GIMPLE dumps do not include everything needed to re-build the GIMPLE. That's also hardly a regression. You could argue in a similar way that you want -vops and -alias to be the default. That said, I'd be happy to see less if (flags & TDF_GIMPLE) in the dumper and have the default be GIMPLE FE syntax for the individual GIMPLE stmts. But I do not have the cycles to make this happen and adjust all of the testsuite fallout.