[PATCH] D145404: [LLVM] Remove support for constant scalable vector GEPs.

[Paul Walker via Phabricator via llvm-commits]

paulwalker-arm added inline comments.


================
Comment at: llvm/test/Transforms/InstSimplify/vscale.ll:198
 ;
-  %ptr = getelementptr <vscale x 4 x i32>, ptr null, i64 %x
+  %ptr = getelementptr <vscale x 4 x i32>, ptr null, i64 1
   ret ptr %ptr
----------------
nikic wrote:
> Any reason for this change?
I wanted to show that even though all operands are constant this form of GEP is still not constant foldable.  I figured this to be a better version of the original test, but I might have missed some subtlety that means an extra test is more appropriate?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145404