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

Owen Anderson via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 7 11:03:18 PDT 2016


resistor added a subscriber: resistor.
resistor added a comment.

I agree with Steve here.  Even code that is built with -ffast-math has to live in a reality where NaNs, Infs, etc. are sometimes generated by external function calls, provided as user input, etc.  It's critical that this code be able to filter out these invalid values, precisely **because** the body of the code will be optimized on the assumption that those values did not need to be considered.


http://reviews.llvm.org/D18513





More information about the llvm-commits mailing list