[PATCH] D19487: [LV] Reallow interleaved load groups with gaps

[silviu.baranga@arm.com via llvm-commits]

sbaranga added a comment.

In http://reviews.llvm.org/D19487#412065, @mssimpso wrote:

> > I might be missing something obvious, but why is TC >= Step if TC % Step is 0?
>
>
> Sorry, I didn't mean for that to read as an implication. The minimum iterations check is supposed to ensure that TC >= Step. We can add the check for TC is not zero, but I think instcombine will probably remove it. What do you think?


Do you mean the emitMinimumIterationCountCheck? Looks like the minimum check is using whatever getOrCreateVectorTripCount returns, so it probably won't work as expected. I think checking for the zero case here would be ok.


http://reviews.llvm.org/D19487