[PATCH] D54121: [FPEnv] Add constrained FCMP intrinsic

[Cameron McInally via Phabricator via llvm-commits]

cameron.mcinally added a comment.

More insight...

There are 12 x86 builtins and only 22 IEEE functions (not 24 as expected) since the following two IEEE functions don't make sense:

  compareSignalingOrdered(a, b)
  compareSignalingUnordered(a, b)

Those would always raise invalid on NaNs (both SNaNs and QNaNs), so are really of no consequence.

That implies the respective x86 builtins, __builtin_ia32_cmpunordxx and __builtin_ia32_cmpordxx, map to compareQuietUnordered(...) and compareQuietOrdered(...).


Repository:
  rL LLVM

https://reviews.llvm.org/D54121