[PATCH] D15706: [Polly] Follow uses to create value MemoryAccesses
Michael Kruse via llvm-commits
llvm-commits at lists.llvm.org
Tue Dec 22 14:34:44 PST 2015
Meinersbur added a comment.
In http://reviews.llvm.org/D15706#315267, @llvm-commits wrote:
> > +; TODO: Because of the phi with 3 incoming blocks, the loop for => outofscop is
>
> > +; not recognized as a loop, but as a non-affine subregion.
>
>
> My guess would be this is because ScalarEvolution cannot create an
> AddRec for %j. PHIs with multiple incoming blocks are generally not a
> problem.
The loop is correctly identified without the %outofscop block. From the region's standpoint, is also entering the region with the same incoming value. %outofscop adds another surrounding loop.
I think I will look at it again and commit this test case separately as it is not specific to this patch; It was helpful for finding a bug, though;
http://reviews.llvm.org/D15706
More information about the llvm-commits
mailing list