[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