[PATCH] D40778: [DebugIR] Revive the Debug IR pass. [Added llvm-commits]

[Vedant Kumar via Phabricator via llvm-commits]

vsk requested changes to this revision.
vsk added a subscriber: Ralith.
vsk added a comment.
This revision now requires changes to proceed.

Thanks for resubmitting this @bollu. I discussed this idea with @Ralith, @rkruppe, @aprantl and others. I have a better notion now of how debugIR could be helpful for optimization and frontend authors.

A few high-level technical comments:

- Should debugIR be a pass? The main use seems to be: there is an existing *.ll file on disk, and it needs to be debugged. Doesn't it make more sense to add a cl::opt to the LL reader to enable debugIR, so that the existing file can be debugged in-place?
- Is there a reason to hand-roll `DebugMetadataRemover` when there's an existing `StripDebugInfo` utility?


https://reviews.llvm.org/D40778