[PATCH] D65975: [NewPM][PassInstrumentation] IR printing support from clang driver

This revision was automatically updated to reflect the committed changes. Closed by commit rL368804: [NewPM][PassInstrumentation] IR printing support from clang driver (authored by twoh, committed by ). Herald added a project: LLVM. Changed prior to commit: https://reviews.llvm.org/D65975?vs=21

[PATCH] D65975: [NewPM][PassInstrumentation] IR printing support from clang driver

fedor.sergeev accepted this revision. fedor.sergeev added a comment. This revision is now accepted and ready to land. LGTM. Thanks for addressing this! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65975/new/ https://reviews.llvm.org/D65975

[PATCH] D65975: [NewPM][PassInstrumentation] IR printing support from clang driver

twoh added a comment. @fedor.sergeev Sorry for the late reply. I missed your comment. I found that with the new pass manager module verifier is not scheduled by default, and AlwaysInlinerPass is the only one scheduled if I provide no -O option. I removed -O2 from the test command and replaced F

[PATCH] D65975: [NewPM][PassInstrumentation] IR printing support from clang driver

twoh updated this revision to Diff 214855. twoh added a comment. Avoid -O2 from the test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65975/new/ https://reviews.llvm.org/D65975 Files: clang/lib/CodeGen/BackendUtil.cpp clang/test/Misc/printer

[PATCH] D65975: [NewPM][PassInstrumentation] IR printing support from clang driver

fedor.sergeev added a comment. In D65975#1621892 , @twoh wrote: > @fedor.sergeev @yamauchi I saw your discussions over llvm-dev mailing list > regarding IR printing with the new pass manager, and though this might be the > reason why IR printing is not s

[PATCH] D65975: [NewPM][PassInstrumentation] IR printing support from clang driver

yamauchi added a comment. In D65975#1621892 , @twoh wrote: > @fedor.sergeev @yamauchi I saw your discussions over llvm-dev mailing list > regarding IR printing with the new pass manager, and though this might be the > reason why IR printing is not suppor

[PATCH] D65975: [NewPM][PassInstrumentation] IR printing support from clang driver

twoh added a comment. @fedor.sergeev @yamauchi I saw your discussions over llvm-dev mailing list regarding IR printing with the new pass manager, and though this might be the reason why IR printing is not supported under new PM with clang. I would appreciate if you can take a look. Repository

[PATCH] D65975: [NewPM][PassInstrumentation] IR printing support from clang driver

twoh created this revision. twoh added reviewers: fedor.sergeev, philip.pfaffe. Herald added a project: clang. Herald added a subscriber: cfe-commits. https://reviews.llvm.org/D50923 enabled the IR printing support for the new pass manager, but only for the case when `opt` tool is used as a drive