pcwang-thead added inline comments.

================
Comment at: clang/include/clang/Basic/riscv_vector.td:1579
+      llvm::Function *F =
+        CGM.getIntrinsic(llvm::Intrinsic::read_register, {Int64Ty});
+      return Builder.CreateCall(F, Metadata);
----------------
craig.topper wrote:
> pcwang-thead wrote:
> > The value type of `read_register` is `i32` in 
> > `llvm/test/CodeGen/RISCV/vlenb.ll`, but I don't insist on it.
> Doesn't this need to match the size of long?
So what about `i32` for rv32 and `i64` for rv64?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141032

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to