From 468b40bcb95976311b1ed4f1b988d186b45dab49 Mon Sep 17 00:00:00 2001 From: James E Wilson Date: Thu, 13 Sep 2007 09:33:49 -0700 Subject: [PATCH] re PR tree-optimization/33389 (Revision 128239 causes libgomp failure) 2007-09-13 James E. Wilson 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 --- gcc/ChangeLog | 6 ++++++ gcc/tree-ssa-operands.c | 9 ++++++++- 2 files changed, 14 insertions(+), 1 deletion(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 7f50b61561b..4ea43d008ea 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2007-09-13 James E. Wilson + + 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 David Ung diff --git a/gcc/tree-ssa-operands.c b/gcc/tree-ssa-operands.c index 3562e712137..5c7a9e0b89d 100644 --- 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; -- 2.30.2