[PATCH] D23854: [LoopAccessAnalysis] Recognize geps that include s/zexts as consecutive memory accesses.

[Balaram Makam via llvm-commits]

bmakam added a comment.

In https://reviews.llvm.org/D23854#525217, @mssimpso wrote:

> Balaram,
>
> Here's a quick question before I start looking at the details. Won't we still have two implementations of isConsecutiveAccess after this change? Is something preventing us from replacing LoadStoreVectorizer::isConsecutiveAccess with llvm::isConsecutiveAccess?


No. I will remove the implementation in LoadStoreVectorizer as a follow up patch if this looks good.


https://reviews.llvm.org/D23854