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

Mikael Holmén via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 3 05:29:41 PST 2023


uabelho added a comment.

In D145051#4166656 <https://reviews.llvm.org/D145051#4166656>, @nikic wrote:

> I think what we can do instead is to mark MemorySSA as preserved if only debuginfo intrinsics have been removed. (This is safe, MSSA explicitly never tracks debuginfo intrinsics.)

Does everyone agree this is the way forward then? If so I can put up a patch doing that on Monday.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145051



More information about the llvm-commits mailing list