[llvm] [RISCV][VLOPT] Compute demanded VLs up front (PR #124530)

Michael Maitland via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 28 07:24:45 PST 2025


================
@@ -1371,18 +1369,8 @@ bool RISCVVLOptimizer::runOnMachineFunction(MachineFunction &MF) {
       if (!tryReduceVL(MI))
         continue;
       MadeChange = true;
-      PushOperands(MI, /*IgnoreSameBlock*/ true);
     }
   }
 
-  while (!Worklist.empty()) {
-    assert(MadeChange);
-    MachineInstr &MI = *Worklist.pop_back_val();
-    assert(isCandidate(MI));
----------------
michaelmaitland wrote:

You are removing the assert that triggered the cross BB failure. Can we add back the VL=1 code above?

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


More information about the llvm-commits mailing list