https://github.com/artagnon closed
https://github.com/llvm/llvm-project/pull/71399
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/topperc approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/71399
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
artagnon wrote:
Gentle ping.
https://github.com/llvm/llvm-project/pull/71399
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
artagnon wrote:
The main takeaways here:
- There is only ever an i64 variant of llrint produced, and we have tested
codegen of that on RV32 and RV64.
- Only an i32 variant of lrint is produced on RV32, and an i64 variant on RV64.
We have tested the codegen of both.
Hence, I would conclude that
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Ramkumar Ramachandra (artagnon)
Changes
Ever since 98c90a1 (ISel: introduce vector ISD::LRINT, ISD::LLRINT; custom
RISCV lowering) landed, there have been several discussions on how the lrint
and llrint libcalls would lower to LLVM IR via
https://github.com/artagnon created
https://github.com/llvm/llvm-project/pull/71399
Ever since 98c90a1 (ISel: introduce vector ISD::LRINT, ISD::LLRINT; custom
RISCV lowering) landed, there have been several discussions on how the lrint
and llrint libcalls would lower to LLVM IR via clang on RV
