[PATCH] D159084: InstCombine: Fold is.fpclass(x, fcInf) to fabs+fcmp

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 30 03:01:32 PDT 2023


arsenm added a comment.

In D159084#4627826 <https://reviews.llvm.org/D159084#4627826>, @SixWeining wrote:

> If ISD::IS_FPCLASS is legal or custom on some targets, do we still need the transform?

Regardless of codegen I think this is the better canonical form. Codegen should implement the reverse transform based on target preference


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

https://reviews.llvm.org/D159084



More information about the llvm-commits mailing list