[PATCH] D133567: [AA] Improve the BasicAA analysis capability

[Allen zhong via Phabricator via llvm-commits]

Allen marked 2 inline comments as done.
Allen added inline comments.


================
Comment at: llvm/lib/Analysis/BasicAliasAnalysis.cpp:615
+        if (isa<ScalableVectorType>(GEPOp->getSourceElementType()) &&
+            (I - GEPOp->op_begin() != GEPOp->getNumOperands() - 1)) {
+          Decomposed.Base = V;
----------------
nikic wrote:
> Instead of checking the source element type, this should check whether the getTypeAllocSize fetched below isScalable.
apply your comment, thanks


================
Comment at: llvm/test/Transforms/GVN/vscale.ll:49
 
 ; TODO: BasicAA return MayAlias for %gep1,%gep2, could improve as MustAlias.
 define i32 @load_clobber_load_gep1(<vscale x 4 x i32>* %p) {
----------------
nikic wrote:
> Drop TODO.
DONE


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

https://reviews.llvm.org/D133567