[llvm] [DAGCombiner] Add generic DAG combine for ISD::PARTIAL_REDUCE_MLA (PR #127083)

Benjamin Maxwell via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 21 03:40:21 PST 2025


================
@@ -836,9 +836,8 @@ void TargetLoweringBase::initActions() {
     setOperationAction(ISD::SET_FPENV, VT, Expand);
     setOperationAction(ISD::RESET_FPENV, VT, Expand);
 
-    // PartialReduceMLA operations default to expand.
-    setOperationAction({ISD::PARTIAL_REDUCE_UMLA, ISD::PARTIAL_REDUCE_SMLA}, VT,
-                       Expand);
+    for (MVT InputVT : MVT::all_valuetypes())
+      setPartialReduceMLAAction(VT, InputVT, Expand);
----------------
MacDue wrote:

Is this still needed? This is preventing the new DAG combine from applying, which means it's untested in this PR. 

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


More information about the llvm-commits mailing list