[PATCH] D89818: [Compile Time] Make it possible to skip redundant debuginst removal

Christopher Tetreault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 20 12:59:26 PDT 2020


ctetreau added a comment.

This patch is basically the same thing we ended up doing in our downstream to resolve the issue. We're seeing it in LoopUnroll, so that's where we are skipping it.

If this problem crops up in other places, we could do something similar there. And if all the callsites have a mitigation, then we can safely remove `RemoveRedundantDbgInstrs` from `MergeBlockIntoPredecessor`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89818



More information about the llvm-commits mailing list