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

[Andy Kaylor via Phabricator via llvm-commits]

andrew.w.kaylor added inline comments.


================
Comment at: lib/IR/IntrinsicInst.cpp:107
+  else if (RoundingArg.equals("LLVM_ROUND_TONEAREST"))
+    return rmToNearest;
+  else if (RoundingArg.equals("LLVM_ROUND_DOWNWARD")) 
----------------
craig.topper wrote:
> You could maybe use a StringSwitch.
That would be perfect, but can I put an llvm_unreachable() in a StringSwitch?


Repository:
  rL LLVM

https://reviews.llvm.org/D27028