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

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 9 13:41:03 PDT 2021


aeubanks added inline comments.


================
Comment at: llvm/include/llvm/IR/PassManager.h:388
+    // Drop anonymous namespace here handle passes such as NoOpModulePass.
+    ClassName.consume_front("(anonymous namespace)::");
     auto PassName = MapClassName2PassName(ClassName);
----------------
IIRC (and I might not be), there was some issue with different compilers providing slightly different names

But those classes can just manually override `name()` anyway, so I don't think this is necessary


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