[llvm] [RISCV] Handle recurrences in RISCVVLOptimizer (PR #151285)

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 11 11:17:04 PDT 2025


================
@@ -1450,9 +1485,10 @@ static bool isSegmentedStoreInstr(const MachineInstr &MI) {
   }
 }
 
-std::optional<MachineOperand>
-RISCVVLOptimizer::checkUsers(const MachineInstr &MI) const {
-  std::optional<MachineOperand> CommonVL;
+bool RISCVVLOptimizer::checkUsers(const MachineInstr &MI) const {
+  if (MI.isPHI() || MI.isFullCopy() || isTupleInsertInstr(MI))
+    return true;
+
   SmallSetVector<MachineOperand *, 8> Worklist;
----------------
topperc wrote:

This name now shadows the new MachineInstr* Worklist in the class. Should we disambiguate them?

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


More information about the llvm-commits mailing list