[PATCH] D36596: [InstCombine] Add a flag to disable LowerDbgDeclare

[Reid Kleckner via Phabricator via llvm-commits]

rnk added a comment.

I rewrote this to use a cl::opt so that users can try this patch out with an -mllvm flag. It should allow them to get more debug info at the cost of possible innacurate debug info when DSE kicks in as described in PR34136. The long-term solution to PR34136 looks like it's going to take a while, so I want to have a workaround that we can offer to Chromium in the meantime. I expect they might actually use it for Linux and Android crash analysis as well as on Windows.

Does that seem reasonable?


https://reviews.llvm.org/D36596