------- 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 <[EMAIL PROTECTED]>
>
> * 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
