[PATCH] D77828: [BasicAA][SVE] Fix aliasGEP for scalable type.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 9 17:39:23 PDT 2020


efriedma added inline comments.


================
Comment at: llvm/lib/Analysis/BasicAliasAnalysis.cpp:1326
+       SrcEleTy2->getVectorIsScalable()))
+    return MayAlias;
+
----------------
I think this still doesn't completely block off all the relevant cases.

DecomposeGEPExpression is recursive, so there are can be multiple levels of GEPs.  The GEP over the scalable vector may not be GEP1; it could be GEP1->getPointerOperand(), or something like that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77828





More information about the llvm-commits mailing list