[PATCH] D106850: [InstCombine] Simplify llvm.vscale when vscale_range attribute exists

[Fraser Cormack via Phabricator via llvm-commits]

frasercrmck added inline comments.


================
Comment at: llvm/lib/Analysis/InstructionSimplify.cpp:5787
+        return nullptr;
+      unsigned MinSVEVectorSize, MaxSVEVectorSize;
+      std::tie(MinSVEVectorSize, MaxSVEVectorSize) = Attr.getVScaleRangeArgs();
----------------
I'd say that `SVE` isn't a good variable name here since RISC-V will also use this attribute at some point.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106850