Patchwork Fix PR53340

login
register
mail settings
Submitter Richard Guenther
Date May 14, 2012, 11:33 a.m.
Message ID <Pine.LNX.4.64.1205141332330.11924@jbgna.fhfr.qr>
Download mbox | patch
Permalink /patch/158968/
State New
Headers show

Comments

Richard Guenther - May 14, 2012, 11:33 a.m.
Boolean logic is hard ... this fixes a bug in the previous re-org
of clean/op_valid_in_sets.

Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.

Richard.

2012-05-14  Richard Guenther  <rguenther@suse.de>

	PR tree-optimization/53340
	* tree-ssa-pre.c (op_valid_in_sets): Fix error in last commit.

Patch

Index: gcc/tree-ssa-pre.c
===================================================================
--- gcc/tree-ssa-pre.c	(revision 187449)
+++ gcc/tree-ssa-pre.c	(working copy)
@@ -2007,8 +2007,8 @@  op_valid_in_sets (bitmap_set_t set1, bit
   if (op && TREE_CODE (op) == SSA_NAME)
     {
       unsigned int value_id = VN_INFO (op)->value_id;
-      if (!bitmap_set_contains_value (set1, value_id)
-	  || (set2 && !bitmap_set_contains_value  (set2, value_id)))
+      if (!(bitmap_set_contains_value (set1, value_id)
+	    || (set2 && bitmap_set_contains_value  (set2, value_id))))
 	return false;
     }
   return true;