[SelectionDAG] Add SDTCisInt<1> to SDTVecReduce and SDTCisEltOfVec to SDTFPVecReduce. NFC

[topperc]

No description provided.

[llvmbot]

@llvm/pr-subscribers-llvm-selectiondag

Author: Craig Topper (topperc)

Changes

---

Full diff: https://github.com/llvm/llvm-project/pull/162761.diff

1 Files Affected:

• (modified) llvm/include/llvm/Target/TargetSelectionDAG.td (+2-2)

diff --git a/llvm/include/llvm/Target/TargetSelectionDAG.td b/llvm/include/llvm/Target/TargetSelectionDAG.td
index 7bc90d4428800..5f4f8c391c489 100644
--- a/llvm/include/llvm/Target/TargetSelectionDAG.td
+++ b/llvm/include/llvm/Target/TargetSelectionDAG.td
@@ -300,10 +300,10 @@ def SDTVecInsert : SDTypeProfile<1, 3, [    // vector insert
   SDTCisEltOfVec<2, 1>, SDTCisSameAs<0, 1>, SDTCisPtrTy<3>
 ]>;
 def SDTVecReduce : SDTypeProfile<1, 1, [    // vector reduction
-  SDTCisInt<0>, SDTCisVec<1>
+  SDTCisInt<0>, SDTCisVec<1>, SDTCisInt<1>
 ]>;
 def SDTFPVecReduce : SDTypeProfile<1, 1, [  // FP vector reduction
-  SDTCisFP<0>, SDTCisVec<1>
+  SDTCisFP<0>, SDTCisVec<1>, SDTCisFP<1>
 ]>;
 
 def SDTVecReverse : SDTypeProfile<1, 1, [  // vector reverse

[s-barannikov]

LGTM

[s-barannikov]

---

Can this be:

Suggested change

	SDTCisFP<0>, SDTCisVec<1>, SDTCisFP<1>
	SDTCisEltOfVec<0, 1>, SDTCisVec<1>, SDTCisFP<1>

Corresponding LLVM intrinsics require the result type to match the source element type, but I don't know if this is still a requirement in SDAG.

[topperc]

It might be a requirement in SelectionDAG for FP but not int.