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

Balaram Makam via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 30 08:36:47 PDT 2016

bmakam added a comment.

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

> The logic looks fine, but I'm wondering if it makes sense to add this functionality directly into SCEV? It might be beneficial to transformations other than SLP and LoopIdiom. What do you think?

Matt, I agree with you in principle but I do not have enough understanding of how SCEV works and after my first time reading through SCEV analysis I do not know how involved it is to add this functionality directly into SCEV.


More information about the llvm-commits mailing list