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

[Nikita Popov via Phabricator via llvm-commits]

nikic 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
----------------
paulwalker-arm wrote:
> 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?
Oh I see, that makes sense.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145404