[PATCH] D23854: [LoopAccessAnalysis] Recognize geps that include s/zexts as consecutive memory accesses.
Balaram Makam via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 25 08:24:07 PDT 2016
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
More information about the llvm-commits
mailing list