[PATCH] D109542: [NPM] Complementary fixes for opt option -print-pipeline-passes

Bjorn Pettersson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 9 14:01:36 PDT 2021


bjope added inline comments.


================
Comment at: llvm/include/llvm/IR/PassManager.h:386
                      function_ref<StringRef(StringRef)> MapClassName2PassName) {
-    auto ClassName = name();
+    StringRef ClassName = name();
+    // Drop anonymous namespace here handle passes such as NoOpModulePass.
----------------
I think this should use `DerivedT::name()` and then I can remove the consume_front below.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109542



More information about the llvm-commits mailing list