[PATCH] D133294: [Assignment Tracking][11/*] Update RemoveRedundantDbgInstrs

[Orlando Cazalet-Hyams via Phabricator via llvm-commits]

Orlando added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/BasicBlockUtils.cpp:391-392
       // iteration).
-      if (!R.second)
-        ToBeRemoved.push_back(DVI);
+      ToBeRemoved.push_back(DVI);
       continue;
     }
----------------
StephenTozer wrote:
> 
I think the continue is correct (this isn't the end of the loop, it's inside the `if (DbgValueInst *DVI = dyn_cast<DbgValueInst>(&I))` block) - `continue` because we're still in a block of consecutive dbg intrinsics.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D133294/new/

https://reviews.llvm.org/D133294