[PATCH] D87216: [NewPM] Support --print-before/after in NPM

[Yuanfang Chen via Phabricator via llvm-commits]

ychen accepted this revision.
ychen added a comment.

It is very unfortunate that we have to manage and translate between two sets of names (one pass name and one type name). This makes me wonder if we just keep the pass name as the return value of PassInfoMixin::name and get rid of class name everywhere. Right now I couldn't think of anything is blocking us from doing that. WDYT?  @asbirlea ?



================
Comment at: llvm/include/llvm/Passes/StandardInstrumentations.h:24
 #include "llvm/IR/ValueHandle.h"
+#include "llvm/Passes/PassBuilder.h"
 #include "llvm/Support/CommandLine.h"
----------------
It is better if forward declaring PassInstrumentationCallbacks could remove this include.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87216