[PATCH] D53216: [FPEnv] Add constrained intrinsics for MAXNUM and MINNUM

Cameron McInally via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 18 11:19:09 PDT 2018


cameron.mcinally added inline comments.


================
Comment at: docs/LangRef.rst:14419
+                                            metadata <rounding mode>,
+                                            metadata <exception behavior>)
+
----------------
A quick note that the rounding mode doesn't apply to these intrinsics, but I suspect that this is really a larger problem. E.g. rint, nearbyint, etc also take a rounding mode argument but never use it. I suggest we handle that problem in an entirely separate patch for all constrained intrinsics at once.


Repository:
  rL LLVM

https://reviews.llvm.org/D53216





More information about the llvm-commits mailing list