[PATCH] D138081: [IR] Split out IR printing passes into IRPrinter

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 17 09:12:06 PST 2022


aeubanks added inline comments.


================
Comment at: llvm/include/llvm/IRPrinter/IRPrintingPasses.h:33
+///
+/// Note: This pass is for use with the new pass manager. Use the create...Pass
+/// functions above to create passes for use with the legacy pass manager.
----------------
tejohnson wrote:
> aeubanks wrote:
> > obsolete (and ditto below)
> @aeubanks is the legacy pass manager support needed at all anymore? I'm not sure of the status there.
we still need legacy PM passes in the codegen pipeline, which for sure includes the IR printer passes


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D138081/new/

https://reviews.llvm.org/D138081



More information about the llvm-commits mailing list