[PATCH] D130445: [DAG] Use recursivelyDeleteUnusedNodes in CommitTargetLoweringOpt.

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:1209
-  // are deleted, make sure to remove them from our worklist.
-  WorklistRemover DeadNodes(*this);
   DAG.ReplaceAllUsesOfValueWith(TLO.Old, TLO.New);
----------------
Why don't we need the WorklistRemove here anymore? The comment applies to what can happen inside of ReplaceAllUsesOfValueWith.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130445