[PATCH] D84772: [NewPM][PassInstrument] Add a new kind of before-pass callback that only get called if the pass is not skipped

[Alina Sbirlea via Phabricator via llvm-commits]

asbirlea added inline comments.


================
Comment at: llvm/include/llvm/IR/PassInstrumentation.h:77
   using BeforePassFunc = bool(StringRef, Any);
+  using BeforePassRunFunc = void(StringRef, Any);
   using AfterPassFunc = void(StringRef, Any);
----------------
aeubanks wrote:
> asbirlea wrote:
> > Perhaps `BeforePassRequiredFunc` (or similar) is more informative that these callbacks are run for required passes?
> "required" is sort of misleading, these callbacks will run on passes that aren't "required" as long as nothing says the pass should be skipped.
> `BeforePassIsNotSkippedFunc`? a little verbose
Sure, in my mind if a pass is not skipped, it can be viewed as required. I'm ok with either renaming as long we make it more informative.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84772