[Bug ipa/61659] [4.9 Regression] Extra undefined symbol because of devirtualization

2014-09-10 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659 --- Comment #36 from Jason Merrill --- Author: jason Date: Wed Sep 10 17:28:59 2014 New Revision: 215134 URL: https://gcc.gnu.org/viewcvs?rev=215134&root=gcc&view=rev Log: PR c++/61659 * decl.c (grokfndecl): Don't set DECL_COMDAT on stat

[Bug ipa/61659] [4.9 Regression] Extra undefined symbol because of devirtualization

2014-09-10 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659 Markus Trippelsdorf changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug ipa/61659] [4.9 Regression] Extra undefined symbol because of devirtualization

2014-09-10 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659 --- Comment #34 from Jason Merrill --- Author: jason Date: Wed Sep 10 14:27:40 2014 New Revision: 215131 URL: https://gcc.gnu.org/viewcvs?rev=215131&root=gcc&view=rev Log: PR lto/53808 PR c++/61659 * decl2.c (note_comdat_fn): New.

[Bug ipa/61659] [4.9 Regression] Extra undefined symbol because of devirtualization

2014-09-09 Thread danglin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659 --- Comment #33 from John David Anglin --- The issue in comment 32 was introduced in revision 214177 .

[Bug ipa/61659] [4.9 Regression] Extra undefined symbol because of devirtualization

2014-09-04 Thread danglin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659 John David Anglin changed: What|Removed |Added CC||danglin at gcc dot gnu.org --- Comme

[Bug ipa/61659] [4.9 Regression] Extra undefined symbol because of devirtualization

2014-09-04 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659 Jason Merrill changed: What|Removed |Added CC|jason at redhat dot com| --- Comment #31 from Jason Merri

[Bug ipa/61659] [4.9 Regression] Extra undefined symbol because of devirtualization

2014-09-04 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659 --- Comment #30 from Jason Merrill --- Author: jason Date: Thu Sep 4 12:37:05 2014 New Revision: 214900 URL: https://gcc.gnu.org/viewcvs?rev=214900&root=gcc&view=rev Log: PR ipa/61659 * decl.c (duplicate_decls): Check DECL_DECLARED_INLI

[Bug ipa/61659] [4.9 Regression] Extra undefined symbol because of devirtualization

2014-09-04 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659 --- Comment #29 from Markus Trippelsdorf --- (In reply to Jason Merrill from comment #28) > (In reply to Markus Trippelsdorf from comment #27) > > Possible fix: > > Applied to trunk, thanks. The fix isn't in trunk yet. Forgot to push?

[Bug ipa/61659] [4.9 Regression] Extra undefined symbol because of devirtualization

2014-09-02 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659 --- Comment #28 from Jason Merrill --- (In reply to Markus Trippelsdorf from comment #27) > Possible fix: Applied to trunk, thanks.

[Bug ipa/61659] [4.9 Regression] Extra undefined symbol because of devirtualization

2014-09-01 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659 --- Comment #27 from Markus Trippelsdorf --- Possible fix: diff --git a/gcc/cp/decl.c b/gcc/cp/decl.c index d03f8a4707e5..d8fb35e21237 100644 --- a/gcc/cp/decl.c +++ b/gcc/cp/decl.c @@ -2215,7 +2215,7 @@ duplicate_decls (tree newdecl, tree oldde

[Bug ipa/61659] [4.9 Regression] Extra undefined symbol because of devirtualization

2014-09-01 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659 --- Comment #26 from Markus Trippelsdorf --- ยง14.7.3/12 says: An explicit specialization of a function template is inline only if it is declared with the inline specifier or defined as deleted, and independently of whether its function template i

[Bug ipa/61659] [4.9 Regression] Extra undefined symbol because of devirtualization

2014-09-01 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659 --- Comment #25 from Markus Trippelsdorf --- (In reply to Jason Merrill from comment #24) > Created attachment 33407 [details] > Patch for 4.9 > > Here's a patch for the 4.9 branch. I'm nervous about applying it because I > suspect that we have

[Bug ipa/61659] [4.9 Regression] Extra undefined symbol because of devirtualization

2014-08-27 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659 --- Comment #24 from Jason Merrill --- Created attachment 33407 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33407&action=edit Patch for 4.9 Here's a patch for the 4.9 branch. I'm nervous about applying it because I suspect that we have

[Bug ipa/61659] [4.9 Regression] Extra undefined symbol because of devirtualization

2014-08-27 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659 Markus Trippelsdorf changed: What|Removed |Added Known to work||5.0 Summary|[4.9/5 Regr