Patchwork Fix a thinko in dumping in ipa-cp.c

login
register
mail settings
Submitter Martin Jambor
Date Feb. 18, 2013, 7:54 p.m.
Message ID <20130218195435.GA19120@virgil.suse>
Download mbox | patch
Permalink /patch/221460/
State New
Headers show

Comments

Martin Jambor - Feb. 18, 2013, 7:54 p.m.
Hi,

in ipa-cp there is a dumping condition that was intended to trigger
for non-aliases and non-thunks but I have lost the negations somewhere
when introducing the code.  This probably shows this particular
dumping is not very crucial but I think it summarizes the
initialization well and decided to keep it.

The following patch has successfully bootstrapped and passed testsuite
on x86_64-linux, I intend to commit it as obvious tomorrow unless
someone objects.

Thanks,

Martin


2013-02-07  Martin Jambor  <mjambor@suse.cz>

	* ipa-cp.c (initialize_node_lattices): Fix dumping condition.

Patch

Index: src/gcc/ipa-cp.c
===================================================================
--- src.orig/gcc/ipa-cp.c
+++ src/gcc/ipa-cp.c
@@ -725,7 +725,7 @@  initialize_node_lattices (struct cgraph_
 	    set_all_contains_variable (plats);
 	}
       if (dump_file && (dump_flags & TDF_DETAILS)
-	  && node->alias && node->thunk.thunk_p)
+	  && !node->alias && !node->thunk.thunk_p)
 	fprintf (dump_file, "Marking all lattices of %s/%i as %s\n",
 		 cgraph_node_name (node), node->uid,
 		 disable ? "BOTTOM" : "VARIABLE");