https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70273
--- Comment #11 from Richard Henderson ---
Author: rth
Date: Mon Mar 21 23:03:56 2016
New Revision: 234386
URL: https://gcc.gnu.org/viewcvs?rev=234386=gcc=rev
Log:
PR c++/70273
* decl.c (notice_forced_label_r): New.
(cp_finish_decl): Use
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70273
Richard Henderson changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70273
Richard Henderson changed:
What|Removed |Added
Attachment #38003|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70273
--- Comment #7 from Richard Biener ---
(In reply to Richard Biener from comment #6)
> (In reply to Jakub Jelinek from comment #5)
> > Or better yet decide whether to walk the initializers or not based on
> > !gimple_body (fndecl) && !fun->cfg ?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70273
--- Comment #2 from Richard Biener ---
Looks like FE constructor cloning "breaks" this by not having the new flag set
before gimplification.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70273
--- Comment #4 from Jakub Jelinek ---
Perhaps has_label_address_in_static_1 could move to cp/optimize.c and be done
next to the tree_versionable_function_p call in there, with appropriate
comment?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70273
Richard Henderson changed:
What|Removed |Added
Assignee|unassigned at gcc dot gnu.org |rth at gcc dot gnu.org
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70273
--- Comment #9 from Richard Henderson ---
Created attachment 38003
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=38003=edit
proposed patch
Alternately, instead of setting local_decls early (and doing
other tri-state-ish things in
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70273
--- Comment #5 from Jakub Jelinek ---
Or better yet decide whether to walk the initializers or not based on
!gimple_body (fndecl) && !fun->cfg ?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70273
Richard Biener changed:
What|Removed |Added
Priority|P3 |P1
Target Milestone|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70273
--- Comment #3 from Jakub Jelinek ---
I think the problem is that after the r234261 change the new flag is set only
during gimplification, which is too late for the C++ ctors and dtors, which
need to check copy_forbidden already before
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70273
Jakub Jelinek changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70273
--- Comment #6 from Richard Biener ---
(In reply to Jakub Jelinek from comment #5)
> Or better yet decide whether to walk the initializers or not based on
> !gimple_body (fndecl) && !fun->cfg ?
or make the flag a tri-state and if we compute it
13 matches
Mail list logo
