This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Fix PR34334, make sure to add virtual operands for the fallback SMT itself


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 */
  };
  


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]