This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix PR34334, make sure to add virtual operands for the fallback SMT itself
- From: Richard Guenther <rguenther at suse dot de>
- To: gcc-patches at gcc dot gnu dot org
- Date: Tue, 4 Dec 2007 16:49:12 +0100 (CET)
- Subject: Re: [PATCH] Fix PR34334, make sure to add virtual operands for the fallback SMT itself
- References: <Pine.LNX.4.64.0712041509530.4148@zhemvz.fhfr.qr>
On Tue, 4 Dec 2007, Richard Guenther wrote:
> Bootstrap and regtest on x86_64-unknown-linux-gnu in progress, I'll
> apply this to mainline later.
It turns out that with the patch after the cfg_cleanup of the
array-prefetch pass an SSA update is required, else we hit
static void
execute_todo (unsigned int flags)
{
#if defined ENABLE_CHECKING
if (need_ssa_update_p ())
gcc_assert (flags & TODO_update_ssa_any);
#endif
on gcc.dg/prefetch-loop-arrays-1.c.
So I'll add the following hunk to the patch.
Richard.
* tree-ssa-loop.c (pass_loop_prefetch): Update SSA form.
Index: tree-ssa-loop.c
===================================================================
*** tree-ssa-loop.c (revision 130600)
--- tree-ssa-loop.c (working copy)
*************** struct tree_opt_pass pass_loop_prefetch
*** 535,541 ****
0, /* properties_provided */
0, /* properties_destroyed */
0, /* todo_flags_start */
! TODO_dump_func | TODO_verify_loops, /* todo_flags_finish */
0 /* letter */
};
--- 535,542 ----
0, /* properties_provided */
0, /* properties_destroyed */
0, /* todo_flags_start */
! TODO_dump_func | TODO_verify_loops
! | TODO_update_ssa, /* todo_flags_finish */
0 /* letter */
};