[PATCH] D88928: [Utils] Skip RemoveRedundantDbgInstrs in MergeBlockIntoPredecessor (PR47746)

Christopher Tetreault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 20 15:50:05 PDT 2020


ctetreau added a comment.

I don't know if this patch as-is exhaustively removes redundant debug instructions in all cases it did before, or if we care, so I can't approve it, but I am OK with this approach.

I would like to see this issue addressed sooner than later though. If this patch is merged, then D89818 <https://reviews.llvm.org/D89818> should be abandoned.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88928



More information about the llvm-commits mailing list