[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