[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:42:02 PST 2023


StephenTozer accepted this revision.
StephenTozer added a comment.
This revision is now accepted and ready to land.

LGTM - I think not deleting debug intrinsics if there would be no other codegen changes is the right call for now as a simplest viable fix for the linked issue; hopefully this doesn't result in intrinsic bloat, but if it does that can be solved separately.


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

https://reviews.llvm.org/D145051



More information about the llvm-commits mailing list