http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52037

--- Comment #11 from Markus Trippelsdorf <markus at trippelsdorf dot de> 
2012-01-30 14:20:15 UTC ---
(In reply to comment #9)
> > only virtuals may need updating here.
> > 
> > Less dangerous (considering other callers) might be to simply adjust
> > the code in inline-transform.
> > 
> > "Obvious" patch for this:
> > 
> 
> Hmm, seems to make sense to me.  Thanks for looking into this!
> Honza

Honza,

BTW I've been using the following patch since August last year:

diff --git a/gcc/ipa-inline-transform.c b/gcc/ipa-inline-transform.c
index 75b8e9d..530fca3 100644
--- a/gcc/ipa-inline-transform.c
+++ b/gcc/ipa-inline-transform.c
@@ -123,6 +123,7 @@ can_remove_node_now_p (struct cgraph_node *node, struct
cgraph_edge *e)
     if ((next->callers && next->callers != e)
        || !can_remove_node_now_p_1 (next))
       return false;
+  gcc_unreachable ();
   return true;
 }

And it never triggered since then...

Reply via email to