diff mbox

Fix ordering issue when removing varpool nodes

Message ID 20150205040235.GF31160@kam.mff.cuni.cz
State New
Headers show

Commit Message

Jan Hubicka Feb. 5, 2015, 4:02 a.m. UTC 
Hi,
this is second part of fix for PR ipa/61548. Before removing an variable, it is better
to remove all its aliases so the constructors can be properly propagated.

Bootstrapped/regtested x86_64-linux.

Honza

	PR ipa/61548
	* ipa.c (symbol_table::remove_unreachable_nodes): Fix ordering issue
	when removing varpool nodes.
diff mbox

Patch

Index: ipa.c
===================================================================
--- ipa.c	(revision 220417)
+++ ipa.c	(working copy)
@@ -597,8 +597,20 @@  symbol_table::remove_unreachable_nodes (
 	     or not.  */
 	  && (!flag_ltrans || !DECL_EXTERNAL (vnode->decl)))
 	{
+	  struct ipa_ref *ref = NULL;
+
+	  /* First remove the aliases, so varpool::remove can possibly lookup
+	     the constructor and save it for future use.  */
+	  while (vnode->iterate_direct_aliases (0, ref))
+	    {
+	      if (file)
+		fprintf (file, " %s/%i", ref->referred->name (),
+			 ref->referred->order);
+	      ref->referring->remove ();
+	    }
 	  if (file)
 	    fprintf (file, " %s/%i", vnode->name (), vnode->order);
+          vnext = next_variable (vnode);
 	  vnode->remove ();
 	  changed = true;
 	}