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

Paul Walker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 7 08:43:51 PST 2023


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



More information about the llvm-commits mailing list