[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