[Debugfiy][WIP] Print the pass name next to the result

[gramanas]

Summary

CheckDebugify now prints the pass name right next to the result of the check.

Does not yet work with the new Pass Manager, hence the WIP.

[vsk]

Thanks for the patch. Comments inline --

tools/opt/Debugify.cpp

195 ↗

(On Diff #146939)

You'll need to update test/DebugInfo/debugify-each.ll to reflect this change.

258 ↗

(On Diff #146939)

Please use clang-format to format your diffs. See:

• https://clang.llvm.org/docs/ClangFormat.html
• https://github.com/llvm-mirror/clang/blob/master/tools/clang-format/clang-format-diff.py

tools/opt/PassPrinters.h

67 ↗

(On Diff #146939)

There isn't support for -debugify-each in the newPM yet, so there's no need to tackle that in this patch. I think we should leave newPM-related changes to a future patch.

67 ↗

(On Diff #146939)

I think 'NameOfWrappedPass' would be a more accurate/descriptive name here.

[gramanas]

• Adress review comments

[vsk]

LGTM. Thanks!

As a follow-up, would you mind fixing the situation with debugify output? Currently we're dumping a lot of information to stdout. Ideally we would only print the result of the pipeline to stdout, so as to not break piping opt's output to other LLVM tools. As a first step, I think all of debugify's warnings and messages should be sent to stderr, and then eventually to a separate file.

[gramanas]

Will do, it's allready in my TODO list!