[llvm] [MachineCombiner][NFC] Split target-dependent patterns (PR #87991)

Min-Yih Hsu via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 8 09:06:03 PDT 2024


================
@@ -1191,10 +1191,9 @@ class TargetInstrInfo : public MCInstrInfo {
   /// faster sequence.
   /// \param Root - Instruction that could be combined with one of its operands
   /// \param Patterns - Vector of possible combination patterns
-  virtual bool
-  getMachineCombinerPatterns(MachineInstr &Root,
-                             SmallVectorImpl<MachineCombinerPattern> &Patterns,
-                             bool DoRegPressureReduce) const;
+  virtual bool getMachineCombinerPatterns(MachineInstr &Root,
+                                          SmallVectorImpl<int> &Patterns,
----------------
mshockwave wrote:

is there any reason to use int rather than unsigned? I don't see any usages of negative numbers in this patch.

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


More information about the llvm-commits mailing list