[PATCH] D18513: Simplify isfinite/isnan/isinf in finite-math-only mode

[Steve Canon via llvm-commits]

scanon added a comment.

Breaking isnan, isinf, etc is a non-starter.  I know it's appealing from a consistent formal model viewpoint, but in practice it breaks a lot of code (this is why we have the call fallbacks for iOS/OSX).

I would be delighted to have a finite-math-safe builtin to avoid the calls, but #2 and #4 both seems reasonable to me.  #4 should probably be done regardless of the resolution of this conversation.


http://reviews.llvm.org/D18513