[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