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

[Cameron McInally via Phabricator via llvm-commits]

cameron.mcinally added a comment.

In https://reviews.llvm.org/D54121#1291755, @craig.topper wrote:

> The X86 builtin story is weird. There should be 9 builtins. I'm not sure how you found 12.


Oh, so those are from the online GCC documentation. In the early days of writing the AVX512 builtins, the GCC AVX512 intrinsics were defined first and were used as the reference to keep the two in sync. Old habit, I suppose.

So it looks like LLVM doesn't have any of the GT/GE intrinsics, but rather uses NLE/NGT. And the extra 9th intrinsic is the general cmp.


Repository:
  rL LLVM

https://reviews.llvm.org/D54121