[PATCH] D134639: [VP][RISCV] Add vp.maxnum and vp.minnum intrinsics and RISC-V support.

[Yeting Kuo via Phabricator via llvm-commits]

fakepaper56 added inline comments.


================
Comment at: llvm/docs/LangRef.rst:18874
+
+Predicated floating-point minimum of two vectors of floating-point values.
+
----------------
craig.topper wrote:
> craig.topper wrote:
> > minimum has a different meaning according to IEEE-754. This is an "IEEE-754 minnum".
> "minnum" not "minimum". "minimum" has a different meaning to IEEE-754
Good point. Done.


================
Comment at: llvm/docs/LangRef.rst:18923
+
+Predicated floating-point maximum of two vectors of floating-point values.
+
----------------
craig.topper wrote:
> "IEEE-754 maxnum"
Done.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134639