re PR tree-optimization/33389 (Revision 128239 causes libgomp failure)

2007-09-13  James E. Wilson  <wilson@specifix.com>

	PR tree-optimization/33389
	* tree-ssa-operands.c (append_vuse): If ann->in_vdef_list true,
	then set build_loads before returning.

From-SVN: r128469

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 7f50b61561ba7cbf1021df3b8e4b9e7e26aba3f0..4ea43d008eaffb33fc8465beb2b7c71015999fab 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,9 @@
+2007-09-13  James E. Wilson  <wilson@specifix.com>
+
+       PR tree-optimization/33389
+       * tree-ssa-operands.c (append_vuse): If ann->in_vdef_list true,
+       then set build_loads before returning.
+
 2007-09-13  Sandra Loosemore  <sandra@codesourcery.com>
            David Ung  <davidu@mips.com>
 

diff --git a/gcc/tree-ssa-operands.c b/gcc/tree-ssa-operands.c
index 3562e71213726d85b40022a55b09e43cf46b9d50..5c7a9e0b89dee6d2cf829f587bdb5e5b0636b729 100644 (file)
--- a/gcc/tree-ssa-operands.c
+++ b/gcc/tree-ssa-operands.c
@@ -1164,8 +1164,15 @@ append_vuse (tree var)
 
       /* Don't allow duplicate entries.  */
       ann = get_var_ann (var);
-      if (ann->in_vuse_list || ann->in_vdef_list)
+      if (ann->in_vuse_list)
        return;
+      else if (ann->in_vdef_list)
+       {
+         /* We don't want a vuse if we already have a vdef, but we must
+            still put this in build_loads.  */
+         bitmap_set_bit (build_loads, DECL_UID (var));
+         return;
+       }
 
       ann->in_vuse_list = true;
       sym = var;