On Thu, Jun 30, 2011 at 2:00 PM, Razya Ladelsky <ra...@il.ibm.com> wrote: > Hi, > > This patch fixes the build failure of gcc spec2006 benchmark. > The change is in gimple_duplicate_sese_tail(), where we need to subtract > 1 from the loop's number of iterations. > The stmt created to change the rhs of the loop's condition, used to be > inserted right after the defining stmt of the rhs (an ssa name). > This requires special handling of different cases of the defining stmt: > 1.gimple_stmt > 2.gimple_phi > 3.default_def > > Instead of handling each case separately, if we insert the new stmt at the > begining of the loop's preheader, we're sure that > it's already been defined. > > Bootstrap and testsuite pass successfully (as autopar is not enabled by > default). > No new regressions when the testsuite is run with autopar enabled. > No new regressions for the run of spec2006 with autopar enabled, and gcc > benchmark now passes successfully.. > > OK for trunk?
+ preheader = loop_preheader_edge(orig_loop)->src; space after loop_preheader_edge. Otherwise ok. Thanks, Richard. > Thanks, > Razya > >