[PATCH] D24557: [Loop Vectorizer] Simplified GEP cloning. NFC.

Adam Nemet via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 15 11:28:20 PDT 2016


anemet added a comment.

In https://reviews.llvm.org/D24557#543887, @mssimpso wrote:

> In https://reviews.llvm.org/D24557#543857, @anemet wrote:
>
> > Sounds like the assert should state that one of the operands needs to be a loop-variant uniform while the others all loop-invariant.  Agreed?
>
>
> The "loop-variant uniform" part sounds strange to me, since IV's don't have to be uniform. Don't we mean: one operand "is an induction variable or consecutive pointer" and all other operands are loop-invariant?


WFM.


Repository:
  rL LLVM

https://reviews.llvm.org/D24557





More information about the llvm-commits mailing list