[PATCH] D50923: [New PM][PassInstrumentation] IR printing support for New Pass Manager

[Philip Pfaffe via Phabricator via llvm-commits]

philip.pfaffe added a comment.

This implementation is modeled unnecessarily close to the legacy behavior. `opt` should just manually add the printer instrumentation if a print option is given. In particular., this shouldn't use the `ShouldPrint*` functions. This way there's no need to involve the PassBuilder API, and you also don't pay for the instrumentation if you don't want to print.


Repository:
  rL LLVM

https://reviews.llvm.org/D50923