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

Roger Ferrer Ibanez via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 10 08:49:05 PST 2020


rogfer01 added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:2066
 };
+static const MCPhysReg ArgVRs[] = {
+  RISCV::V16, RISCV::V17, RISCV::V18, RISCV::V19, RISCV::V20,
----------------
Thanks a lot for the patch @HsiangKai !

I would add a comment here, to avoid confusion, stating that this is an interim calling convention and it may be changed in the future.


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