This is the mail archive of the
gcc-bugs@gcc.gnu.org
mailing list for the GCC project.
[Bug tree-optimization/33389] [4.3 Regression] Revision 128239 causes libgomp failure
- From: "hjl at lucon dot org" <gcc-bugzilla at gcc dot gnu dot org>
- To: gcc-bugs at gcc dot gnu dot org
- Date: 13 Sep 2007 00:13:57 -0000
- Subject: [Bug tree-optimization/33389] [4.3 Regression] Revision 128239 causes libgomp failure
- References: <bug-33389-682@http.gcc.gnu.org/bugzilla/>
- Reply-to: gcc-bugzilla at gcc dot gnu 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