[PATCH] D92616: [NewPM] Make pass adaptors less templatey

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks marked an inline comment as done.
aeubanks added inline comments.


================
Comment at: llvm/lib/Passes/StandardInstrumentations.cpp:476
 void PrintIRInstrumentation::printBeforePass(StringRef PassID, Any IR) {
-  if (PassID.startswith("PassManager<") || PassID.contains("PassAdaptor<"))
+  if (isIgnored(PassID))
     return;
----------------
dexonsmith wrote:
> Could / should these changes to use `isIgnored` be committed separately / ahead of time?
it is related to this change but can be done ahead of time: https://reviews.llvm.org/D92625


================
Comment at: llvm/lib/Transforms/Scalar/LoopPassManager.cpp:89
 }
+} // namespace llvm
+
----------------
dexonsmith wrote:
> Can adding this comment be pulled out to a separate commit? It seems unrelated to this change.
this seems fairly benign to me, but sure


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92616