[PATCH] D137811: InstCombine: Perform basic isnan combines on llvm.is.fpclass

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

In D137811#4001711 <https://reviews.llvm.org/D137811#4001711>, @foad wrote:

>> is.fpclass(x, qnan|snan) -> fcmp uno x, 0.0
>> is.fpclass(nnan x, qnan|snan|other) -> is.fpclass(x, other)
>
> The first one sounds good. Do you have a specific motivation for the second one?

Eventually all the class-like tests should be merged into one class call. We still want to reduce the number of tests to perform in the end and also don't want to be ordering dependent


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

https://reviews.llvm.org/D137811