[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