[PATCH] D93013: [RISCV] Define vadd intrinsics and lower to V instructions.

Kito Cheng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 10 05:42:45 PST 2020


kito-cheng added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:2295
+  assert((!UseGPRForF16_F32 || !UseGPRForF64 || LocVT == XLenVT ||
+          (TLI->getSubtarget().hasStdExtV() && ValVT.isScalableVector())) &&
          "Expected an XLenVT at this stage");
----------------
I would suggest add a new assert here, the assertion message seems not fit for the extra checking.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93013



More information about the llvm-commits mailing list