[PATCH] D18639: Use __builtin_isnan/isinf/isfinite in complex

[Hal Finkel via cfe-commits]

hfinkel added a comment.

In http://reviews.llvm.org/D18639#472010, @chandlerc wrote:

> I'm fine with this change, but we should also get Steve to comment on it, and make sure we have a good way of explaining this to users.
>
> In particular, we probably need some documentation around these fast routines that clearly indicates they should only be used when optimizations such as `-ffast-math` and `-ffinite-math-only` should strip the checks. We want to be careful to only use the optimizable forms when that behavior is appropriate. And auditing the ones you've switched for that is something Steve might be better suited to do...


Also, I spoke to Marshall offline about this last week, and I expect he'll also comment.


http://reviews.llvm.org/D18639