[PATCH] D65303: [BPI] Adjust the probability for floating point unordered comparison
Steve Canon via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 8 09:04:46 PDT 2019
scanon added a comment.
It's not that NaN is rare in normal programs, or that it indicates a bug in the code. It's that testing for NaN is usually an indication that you're testing for an exceptional case, and it makes sense to move those off the hot path (i.e. NaN is actually pretty common, but the likelihood of handling it on the normal-value path through code is small).
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65303/new/
https://reviews.llvm.org/D65303
More information about the llvm-commits
mailing list