[PATCH] D90545: [NewPM] Don't run before pass instrumentation on required passes

[Alina Sbirlea via Phabricator via llvm-commits]

asbirlea accepted this revision.
asbirlea added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/include/llvm/IR/PassInstrumentation.h:186
+    if (!isRequired(Pass)) {
+      for (auto &C : Callbacks->BeforePassCallbacks)
+        ShouldRun &= C(Pass.name(), llvm::Any(&IR));
----------------
Could you add a doxygen comment that the `BeforePassCallbacks` are only called if the pass is not required around the declaration of `BeforePassCallbacks`? 
Would be useful to have comments on each of the vectors of callbacks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90545