[PATCH] D133844: [AA] Improve the BasicAA analysis capability base on GEP

Allen zhong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 11 01:33:34 PDT 2022


Allen planned changes to this revision.
Allen added a comment.

In D133844#3844770 <https://reviews.llvm.org/D133844#3844770>, @nikic wrote:

> I don't get how this patch is supposed to work. It seems to create a VariableGEPIndex -- but I don't see the vscale factor represented anywhere in there?
>
> Worth noting that https://reviews.llvm.org/D134648 plans to make vscale non-constant, so making the use of vscale constant expressions is likely not going to be possible going forward...

Thank @nikic for your guidance. It looks a little more complicated than I expected. Yes, this need more work as the vscale is non-constant in the whole project.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D133844/new/

https://reviews.llvm.org/D133844



More information about the llvm-commits mailing list