On Thu, Oct 5, 2017 at 3:16 PM, Bin Cheng <bin.ch...@arm.com> wrote: > Hi, > Function rename_variables_in_bb skips renaming PHI nodes in loop nest if the > outer loop has only one inner loop. This breaks loop nest distribution when > inner loop has PHI node initialized from outer loop's variable. > Unfortunately, > I lost the original C code illustrating the issue. Now it is only triggered > in building spec2006/416.gamess with loop nest distribution, but I failed to > reduce a test from it.
Bah, can you re-try isolating a testcase? > Bootstrap and test in patch set on x86_64 and AArch64, is it OK? Ok. Note that the renaming is a very fragile bit of code - you may run into issues when applying it on loop nests ;) Richard. > Thanks, > bin > 2017-10-04 Bin Cheng <bin.ch...@arm.com> > > * tree-vect-loop-manip.c (rename_variables_in_bb): Rename PHI nodes > when copying loop nest with only one inner loop.