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

Serge Pavlov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 19 09:15:02 PDT 2021


sepavloff added a comment.

Thanks!

I will wait a couple of days, in case some reviewers have additional notes.

In D108037#2952881 <https://reviews.llvm.org/D108037#2952881>, @sivachandra wrote:

> Also, I would like to understand why we cannot use `fxam` for all cases. For, exceptions can be enabled and disabled from software and should not affect the behavior of `isnan`.

For `x86_fp80` it is used in all cases. For `double` and `float` there are more compact and faster solutions.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108037



More information about the llvm-commits mailing list