[PATCH] D83498: [NFC] Derive from PassInfoMixin for no-op/printing passes

[Arthur Eubanks via Phabricator via llvm-commits]

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


================
Comment at: llvm/lib/Passes/PassBuilder.cpp:300
 
-namespace {
+namespace llvm {
 
----------------
ychen wrote:
> How about keeping this local? These are only for testing.
Do you mean keeping this in an anonymous namespace?
As mentioned in the commit, that makes the printed name messed up.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83498