[clang] [llvm] [AArch64] Implement intrinsics for SVE FAMIN/FAMAX (PR #99042)

Paul Walker via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 28 08:20:04 PDT 2024


================
@@ -135,6 +135,8 @@ enum NodeType : unsigned {
   UDIV_PRED,
   UMAX_PRED,
   UMIN_PRED,
+  FAMAX_PRED,
+  FAMIN_PRED,
----------------
paulwalker-arm wrote:

Yep, the latter. We use it here because the ISD node exists for other reasons and so was convenient to reuse here.

In general the main reason to create `FAMAX_PRED` is when there's optimisation to be had.  In this instance I think it's more likely for us break the operation into separate FABS_PRED/FMIN_PRED nodes to let the combiner at it, given it'll be easy to reconstruct the `famin` during isel.

https://github.com/llvm/llvm-project/pull/99042


More information about the cfe-commits mailing list