[PATCH] D147347: [RISCV] Use tail agnostic policy more often when lowering insert_subvector

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:6589
     unsigned Policy = RISCVII::TAIL_UNDISTURBED_MASK_UNDISTURBED;
-    if (VecVT.isFixedLengthVector() &&
-        OrigIdx + 1 == VecVT.getVectorNumElements())
+    if (VecVT.isFixedLengthVector() && EndIndex >= VecVT.getVectorNumElements())
       Policy = RISCVII::TAIL_AGNOSTIC;
----------------
Can EndIndex ever be > than VecVT.getVectorNumElements()?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147347