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