[PATCH] D145051: [ADCE] Only remove debug intrinsics if non debug instructions are removed

Stephen Tozer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 2 01:27:20 PST 2023


StephenTozer added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/ADCE.cpp:572
 
-  for (Instruction *&I : Worklist) {
-    ++NumRemoved;
-    I->eraseFromParent();
+  // Only actully remove the dead instructions if we know we will invalidate
+  // cached analysis results. If we only found debug intrinsics to remove and
----------------



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

https://reviews.llvm.org/D145051



More information about the llvm-commits mailing list