[llvm] [ARM] Speedups for CombineBaseUpdate. (PR #129725)
    Peter Smith via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Wed Mar  5 03:31:54 PST 2025
    
    
  
================
@@ -16284,27 +16299,19 @@ static SDValue CombineBaseUpdate(SDNode *N,
       unsigned NewConstInc = UserOffset - Offset;
       SDValue NewInc = DCI.DAG.getConstant(NewConstInc, SDLoc(N), MVT::i32);
       BaseUpdates.push_back({User, NewInc, NewConstInc});
+      if (BaseUpdates.size() > MaxBaseUpdates)
----------------
smithp35 wrote:
Being pedantic, wouldn't this take BaseUpdates one over MaxBaseUpdates? Not important in practice, but possibly worth making this == MaxBaseUpdates.
https://github.com/llvm/llvm-project/pull/129725
    
    
More information about the llvm-commits
mailing list