[PATCH] D69562: Mapping of FP operations to constrained intrinsics

[Kevin P. Neal via Phabricator via llvm-commits]

kpn added inline comments.


================
Comment at: llvm/lib/IR/FloatingPoint.cpp:170
+        IID = Intrinsic::experimental_constrained_trunc;
+        break;
+      default:
----------------
Can you please update docs/AddingConstrainedIntrics.rst as well? When new intrinsics are added this switch here would need to be updated as well. Might as well document that fact.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69562