[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-05 Thread dje at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 --- Comment #25 from David Edelsohn --- Author: dje Date: Sat Nov 5 13:06:08 2016 New Revision: 241871 URL: https://gcc.gnu.org/viewcvs?rev=241871&root=gcc&view=rev Log: 2016-11-05 Richard Biener PR bootstrap/78188 * g++.dg/

[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-04 Thread dje at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 --- Comment #24 from David Edelsohn --- Author: dje Date: Fri Nov 4 23:20:50 2016 New Revision: 241863 URL: https://gcc.gnu.org/viewcvs?rev=241863&root=gcc&view=rev Log: PR bootstrap/78188 PR c++/71848 * ipa-comdats.c (pass_ipa_comdats::gate):

[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-04 Thread rguenther at suse dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 --- Comment #23 from rguenther at suse dot de --- On November 4, 2016 8:09:15 PM GMT+01:00, "dje at gcc dot gnu.org" wrote: >https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 > >--- Comment #22 from David Edelsohn --- >There are two levels of

[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-04 Thread dje at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 --- Comment #22 from David Edelsohn --- There are two levels of set_comdat_group(). I am going to move the assert to cgraph.h and try to find what else is setting comdat groups. Do you want me to gate IPA comdat in the interim?

[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-04 Thread rguenther at suse dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 --- Comment #21 from rguenther at suse dot de --- On November 4, 2016 3:30:55 PM GMT+01:00, "dje at gcc dot gnu.org" wrote: >https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 > >--- Comment #18 from David Edelsohn --- >Changing pass_ipa_comdat

[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-04 Thread dje at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 --- Comment #20 from David Edelsohn --- Using set_comdat_group(NULL) in varasm.c did not correct the testsuite failures. The only option that has worked so far is to disable ipa-comdat at gate function.

[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-04 Thread dje at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 --- Comment #19 from David Edelsohn --- I will try if (HAVE_COMDAT_GROUP) symbol->set_comdat_group (comdat_group); else symbol->set_comdat_group (NULL); in varasm.c:make_decl_one_only().

[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-04 Thread dje at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 --- Comment #18 from David Edelsohn --- Changing pass_ipa_comdats::gate to return HAVE_COMDAT_GROUP && optimize; does not experience the tree-vrp.c bootstrap failure and does not generate the numerous additional testsuite failures. There is on

[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-04 Thread dje at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 --- Comment #17 from David Edelsohn --- ld: 0711-317 ERROR: Undefined symbol: .std::allocator::allocator() ld: 0711-317 ERROR: Undefined symbol: .std::__cxx11::basic_string, std::allocator >::basic_string(char const*, std::allocator const&) ld: 0

[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-04 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 --- Comment #16 from Richard Biener --- (In reply to David Edelsohn from comment #15) > Another option is to force the COMDAT group to NULL in set_comdat_group() if > !HAVE_COMDAT_GROUP. That would allow the rest of the COMDAT functionality to >

[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-03 Thread dje at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 --- Comment #15 from David Edelsohn --- Another option is to force the COMDAT group to NULL in set_comdat_group() if !HAVE_COMDAT_GROUP. That would allow the rest of the COMDAT functionality to continue to work. Does that make any sense to try?

[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-03 Thread dje at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 --- Comment #14 from David Edelsohn --- Honza suggested Index: varasm.c === --- varasm.c(revision 241793) +++ varasm.c(working copy) @@ -6036,7 +6036,8 @@ #ifdef MAKE_DECL_

[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-03 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 --- Comment #13 from Richard Biener --- Testcase is alongside int a; static void __attribute__((noinline)) foo () { a = 1; } static void __attribute__((noinline)) foo2 () { a = 2; } struct X { virtual void bar (int i) { if (!i) { foo (); __bu

[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-03 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 --- Comment #12 from Richard Biener --- Breakpoint 8, symtab_node::set_comdat_group (this=0x2f05ea10, group=) at /space/rguenther/src/gcc-git/gcc/cgraph.h:218 218|| DECL_P (group)); (gdb) p this->decl

[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-03 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 Richard Biener changed: What|Removed |Added CC||hubicka at gcc dot gnu.org --- Comment

[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-03 Thread dje at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 --- Comment #10 from David Edelsohn --- The get_create() change does change the section_type_conflict on AIX.

[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-03 Thread dje at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 --- Comment #9 from David Edelsohn --- Any difference if you force more GC? AIX has a different process address space layout and has much more aggressive memory reclamation in malloc/free.

[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-03 Thread dje at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 --- Comment #8 from David Edelsohn --- Reading specs from /edelsohn/RICHI/./prev-gcc/specs COLLECT_GCC=/edelsohn/RICHI/./prev-gcc/xg++ Target: powerpc-ibm-aix7.2.0.0 Configured with: /nasfarm/edelsohn/src/sandbox/configure --disable-werror --enab

[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-03 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 --- Comment #7 from Richard Biener --- Same w/o -g. #2 0x014f17a3 in rs6000_xcoff_declare_object_name (file=0x25634c0, name=0x2cd771b0 "*LDFCM..0", decl=) at /space/rguenther/src/gcc-git/gcc/config/rs6000/rs6000.c:35418 354

[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-03 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 --- Comment #6 from Richard Biener --- Can you also share -v output of the failing command? With just -O2 -g and a cross from x86_64-linux to powerpc-ibm-aix7.1 I get obj-ppc-aix/gcc> ./cc1plus -quiet t.ii -O2 -g /nasfarm/edelsohn/src/sandbox/

[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-03 Thread dje at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 --- Comment #5 from David Edelsohn --- Created attachment 39953 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=39953&action=edit Pre-processed tree-ssa-sccvn.c

[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-03 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 --- Comment #4 from Richard Biener --- So the error message at hand is /* Sanity check user variables for flag changes. */ if (sect->named.decl != NULL && DECL_P (sect->named.decl) && decl != sect

[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-03 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 Richard Biener changed: What|Removed |Added Target Milestone|--- |7.0 --- Comment #3 from Richard Biener

[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-02 Thread dje at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 --- Comment #2 from David Edelsohn --- I built the GCC stage1 with and without the tree-vrp.c patch. Same sources, same revision, same directory. cc1plus.good and cc1plus.bad. Then I used both versions of cc1plus to compile tree-ssa-sccvn.c wit

[Bug bootstrap/78188] [7 Regression] AIX Bootstrap broken by tree-vrp.c change

2016-11-02 Thread dje at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78188 David Edelsohn changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|