[PATCH] D148519: [RISCV] Support vector strict rounding operations.
Yeting Kuo via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 17 18:04:30 PDT 2023
fakepaper56 added inline comments.
================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:2572
+
+ // To catch overflow exception, use target strict operations when converting
+ // to integer.
----------------
craig.topper wrote:
> craig.topper wrote:
> > There shouldn't be an overflow exception. If the value is already an "integer" in the FP register we shouldn't go through the integer conversion. The integer conversion only exists because of an oddity of the RISC-V ISA. That's not part of the definition of these functions.
> >
> > The one thing we need to do that the non-strict doesn't handle is converting snan to qnan.
> And we need to avoid raising an inexact exception for nearbyint.
> The one thing we need to do that the non-strict doesn't handle is converting snan to qnan.
Sorry, I don't know why we need to covert snan to qnan? Could you elaborate it ?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D148519/new/
https://reviews.llvm.org/D148519
More information about the llvm-commits
mailing list