[PATCH] D78908: [RISCV] Add patterns for checking isnan

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 30 03:37:14 PDT 2020


asb accepted this revision.
asb added a comment.
This revision is now accepted and ready to land.

LGTM, thanks!



================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoD.td:316
                  1)>;
+def : Pat<(setuo FPR64:$rs1, FPR64:$rs1),
+          (SLTIU (FEQ_D $rs1, $rs1), 1)>;
----------------
lenary wrote:
> asb wrote:
> > I think this pattern and the matching one in RISCVInstrInfoF.td is left untested?
> This pattern is what is tested by `fcmp uno {float,double} %0, 0.000000e+00`. The `sltiu` is shown in the assembly output as `seqz`.
Doh - thanks!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D78908/new/

https://reviews.llvm.org/D78908





More information about the llvm-commits mailing list