[PATCH] D90817: [RISCV] Use the 'si' lib call for (double (fp_to_sint/uint i32 X)) when F extension is enabled.
Luís Marques via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Nov 5 01:57:44 PST 2020
luismarques added a comment.
In D90817#2375613 <https://reviews.llvm.org/D90817#2375613>, @luismarques wrote:
> LGTM. I guess the old libcall was producing the correct result anyway, since the in-register representation is the same?
Actually that's not the case when the FP result doesn't fit an i32. Still waiting for the coffee to kick in!
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D90817/new/
https://reviews.llvm.org/D90817
More information about the llvm-commits
mailing list