[PATCH] D27028: Add intrinsics for constrained floating point operations

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 10 17:47:29 PST 2017


arsenm added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp:928-935
+static bool isStrictFPOp(SDNode *Node) {
+  unsigned Opc = Node->getOpcode();
+  if (Opc == ISD::STRICT_FADD || Opc == ISD::STRICT_FSUB ||
+      Opc == ISD::STRICT_FMUL || Opc == ISD::STRICT_FDIV ||
+      Opc == ISD::STRICT_FREM)
+    return true;
+  return false;
----------------
You can replace this with a switch strict->regular op mapping, and then you avoid having to switch over these again in MutateStrictFPToFP


Repository:
  rL LLVM

https://reviews.llvm.org/D27028





More information about the llvm-commits mailing list