[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