https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80294
Markus Trippelsdorf changed:
What|Removed |Added
Keywords||ice-on-valid-code
Summa
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80294
Richard Biener changed:
What|Removed |Added
Priority|P3 |P2
Target Milestone|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80294
--- Comment #6 from Markus Trippelsdorf ---
(gdb) l
1704/* FIXME speed this up, it's taking 16% of compile time on sieve
testcase. */
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80294
Jakub Jelinek changed:
What|Removed |Added
CC||jakub at gcc dot gnu.org,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80294
--- Comment #8 from Jakub Jelinek ---
ICE started with r218220.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80294
Markus Trippelsdorf changed:
What|Removed |Added
CC||nathan at gcc dot gnu.org
--- Comm
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80294
--- Comment #10 from Markus Trippelsdorf ---
I've posted the simple patch here:
https://gcc.gnu.org/ml/gcc-patches/2017-04/msg00132.html
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80294
--- Comment #11 from Markus Trippelsdorf ---
(In reply to Markus Trippelsdorf from comment #9)
> This was added as an PR79141 fix. CCing Nathan.
Sorry, wrong PR (too many tabs open).
I meant PR70393.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80294
--- Comment #12 from Jakub Jelinek ---
r234636 just moved that, the constructor_elt ce = { index, NULL_TREE }; being
pushed had there been before Nathan's patch too.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80294
--- Comment #13 from Markus Trippelsdorf ---
(In reply to Jakub Jelinek from comment #12)
> r234636 just moved that, the constructor_elt ce = { index, NULL_TREE };
> being pushed had there been before Nathan's patch too.
But checking for valid e
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80294
--- Comment #14 from Jakub Jelinek ---
I don't know what it means to have NULL elt and it looks weird to me, but I'm
not a C++ FE maintainer, I assume Jason and/or Nathan will review your patch
and know what it means.
Just grepping through FOR_EA
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80294
--- Comment #15 from Markus Trippelsdorf ---
I only looked at a small random sample. So yes, you are right and I retract my
patch, because the majority doesn't expect a NULL elt.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80294
--- Comment #16 from Jakub Jelinek ---
It can be context-specific, NULL elts valid only in some CONSTRUCTORs, e.g.
during constexpr processing or whatever. In any case it would be good to
understand if it is intentional and what it means.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80294
Jason Merrill changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|unassigned a
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80294
--- Comment #17 from Jason Merrill ---
Author: jason
Date: Tue Apr 11 21:07:32 2017
New Revision: 246858
URL: https://gcc.gnu.org/viewcvs?rev=246858&root=gcc&view=rev
Log:
PR c++/80294 - ICE with constexpr and inheritance.
* con
15 matches
Mail list logo