================ @@ -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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits