[PATCH] D96972: [RISCV] Support insertion of misaligned subvectors

[Fraser Cormack via Phabricator via llvm-commits]

frasercrmck marked an inline comment as done.
frasercrmck added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:2265
+  SDLoc DL(Op);
+  auto OrigIdx = Op.getConstantOperandVal(2);
+  const RISCVRegisterInfo *TRI = Subtarget.getRegisterInfo();
----------------
craig.topper wrote:
> Should this be unsigned instead of auto? I think getConstantOperandVal returns uint64_t but the rest of this code uses unsigned. I probably should have asked this question in the EXTRACT_SUBVECTOR patch.
Yes, sounds good. Thanks for spotting that. The EXTRACT_SUBVECTOR code is using `unsigned`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96972