[Bug tree-optimization/33389] [4.3 Regression] Revision 128239 causes libgomp failure

[hjl at lucon dot org]

------- Comment #23 from hjl at lucon dot org  2007-09-13 00:13 -------
(In reply to comment #22)
>
> This works for the testcase, but has had no other testing as yet.
> 
> 2007-09-12  James E. Wilson  <wilson@specifix.com>
> 
>          * tree-ssa-operands.c (append_vuse): If ann->in_vdef_list true,
>          then set build_loads before returning.
> 
> Index: tree-ssa-operands.c
> ===================================================================
> --- tree-ssa-operands.c (revision 128394)
> +++ tree-ssa-operands.c (working copy)
> @@ -1164,8 +1164,15 @@
> 
>         /* 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;
> 

It works for C tests in libgomp. I am bootstrapping on Linux/ia64 now. Thanks.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33389