[PATCH] D108037: [X86] Implement llvm.isnan(x86_fp80) as unordered comparison

[Kevin P. Neal via Phabricator via llvm-commits]

kpn added a comment.

I'm not sure I like the idea of having the compiler know exceptions may be enabled change the results of isnan(). It's subtle and can result in surprising behavior if, for example, a function that called isnan() is inlined into a function that uses the constrained intrinsics with "maytrap" or "strict".


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D108037/new/

https://reviews.llvm.org/D108037