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

[Matthew Simpson via llvm-commits]

mssimpso added a comment.

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?


https://reviews.llvm.org/D23854