chandlerc added a reviewer: scanon.
chandlerc added a comment.

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...


http://reviews.llvm.org/D18639



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to