[PATCH] D122678: [RISCV] Add support for vp.fptosi where the result is a mask type.

[Fraser Cormack via Phabricator via llvm-commits]

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

LGTM, thanks. I suspect the patterns will clash with those in D122743 <https://reviews.llvm.org/D122743> but they should be functionally equivalent.



================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:6229
+        // First convert to the same size integer, then convert to mask by via
+        // and+setcc.
+        assert(SrcEltSize >= 16 && "Unexpected FP type!");
----------------
comment's out of date: no more `and`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122678