================
@@ -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

Reply via email to