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

Andy Kaylor via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 5 14:51:07 PST 2016


andrew.w.kaylor added a comment.

I see.  I still think it's worth adding UnreachableDefault() instead of just a comment explaining that other values will assert/crash.  I'm checking the legal values in the verfifier, so this shouldn't be an issue.


Repository:
  rL LLVM

https://reviews.llvm.org/D27028





More information about the llvm-commits mailing list