https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659
bin.cheng amker.cheng at gmail dot com changed:
What|Removed |Added
CC||amker.cheng at
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659
--- Comment #19 from Jason Merrill jason at gcc dot gnu.org ---
Author: jason
Date: Wed Jul 30 17:27:20 2014
New Revision: 213308
URL: https://gcc.gnu.org/viewcvs?rev=213308root=gccview=rev
Log:
PR c++/61659
PR c++/61687
Revert:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659
--- Comment #19 from Jason Merrill jason at gcc dot gnu.org ---
Author: jason
Date: Wed Jul 30 17:27:20 2014
New Revision: 213308
URL: https://gcc.gnu.org/viewcvs?rev=213308root=gccview=rev
Log:
PR c++/61659
PR c++/61687
Revert:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659
--- Comment #18 from Jason Merrill jason at gcc dot gnu.org ---
Author: jason
Date: Wed Jul 30 17:27:14 2014
New Revision: 213307
URL: https://gcc.gnu.org/viewcvs?rev=213307root=gccview=rev
Log:
PR lto/53808
PR c++/61659
* pt.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659
--- Comment #20 from Jason Merrill jason at gcc dot gnu.org ---
Author: jason
Date: Wed Jul 30 21:29:25 2014
New Revision: 213311
URL: https://gcc.gnu.org/viewcvs?rev=213311root=gccview=rev
Log:
PR lto/53808
PR c++/61659
* pt.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659
Jakub Jelinek jakub at gcc dot gnu.org changed:
What|Removed |Added
Target Milestone|4.9.1 |4.9.2
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659
--- Comment #16 from Jason Merrill jason at gcc dot gnu.org ---
Author: jason
Date: Thu Jul 10 19:29:59 2014
New Revision: 212436
URL: https://gcc.gnu.org/viewcvs?rev=212436root=gccview=rev
Log:
PR c++/61659
PR c++/61687
gcc/c-family/
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659
--- Comment #12 from Markus Trippelsdorf trippels at gcc dot gnu.org ---
Well, if I revert Jason's patch it compiles just fine.
gcc-4.9 is also fine. And -fno-devirtualize, too.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659
--- Comment #13 from Markus Trippelsdorf trippels at gcc dot gnu.org ---
Adding #include llvm/CodeGen/GCStrategy.h to
llvm/lib/CodeGen/MachineFunctionAnalysis.cpp fixes the issue.
Can you take care of this Rafael?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659
--- Comment #14 from Jason Merrill jason at gcc dot gnu.org ---
Right. My patch causes us to synthesize ~I so that it's available for
devirtualization, which we weren't doing before. This is allowed by the C++
standard: 3.2p3 says A virtual
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659
--- Comment #15 from Rafael Avila de Espindola rafael.espindola at gmail dot
com ---
(In reply to Jason Merrill from comment #14)
Right. My patch causes us to synthesize ~I so that it's available for
devirtualization, which we weren't doing
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659
Markus Trippelsdorf trippels at gcc dot gnu.org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659
--- Comment #4 from Rafael Avila de Espindola rafael.espindola at gmail dot
com ---
I ran the testcase with just -O2 (original code with -O3, but the reduced
testcase with -O2).
getOption will be part of the vtable, but it can end up being
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659
Jakub Jelinek jakub at gcc dot gnu.org changed:
What|Removed |Added
CC||jakub at gcc dot
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659
Jan Hubicka hubicka at gcc dot gnu.org changed:
What|Removed |Added
Status|NEW |ASSIGNED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659
Jan Hubicka hubicka at gcc dot gnu.org changed:
What|Removed |Added
CC||jason at redhat
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659
--- Comment #8 from Jason Merrill jason at gcc dot gnu.org ---
Author: jason
Date: Mon Jun 30 20:20:55 2014
New Revision: 212174
URL: https://gcc.gnu.org/viewcvs?rev=212174root=gccview=rev
Log:
PR c++/61659
PR lto/53808
gcc/cp
*
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659
--- Comment #9 from Markus Trippelsdorf trippels at gcc dot gnu.org ---
Unfortunately llvm still fails to build:
FAILED: /var/tmp/gcc_test/usr/local/bin/g++ -DGTEST_HAS_RTTI=0
-DLLVMCodeGen_EXPORTS -D_GNU_SOURCE -D__STDC_CONSTANT_MACROS
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659
--- Comment #10 from Markus Trippelsdorf trippels at gcc dot gnu.org ---
Created attachment 33036
-- https://gcc.gnu.org/bugzilla/attachment.cgi?id=33036action=edit
testcase
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659
--- Comment #11 from Andrew Pinski pinskia at gcc dot gnu.org ---
(In reply to Markus Trippelsdorf from comment #10)
Created attachment 33036 [details]
testcase
I think this is a correct error message. See
20 matches
Mail list logo