ychen added inline comments.

================
Comment at: llvm/include/llvm/Passes/PassBuilder.h:626
+  /// typically run at the very beginning.
+  void runRegisteredO0EPCallbacks(ModulePassManager &MPM, bool DebugLogging);
+
----------------
invokeO0EPCallbacks? I saw there is `PassBuilder::invokePeepholeEPCallbacks`.


================
Comment at: llvm/lib/Passes/PassBuilder.cpp:1659
+                                             bool DebugLogging) {
+  for (auto &C : PipelineStartEPCallbacks)
+    C(MPM);
----------------
What I have in mind is a newly added `O0EPCallbacks` field in PassBuilder 
class. Then we can keep existing EPCallbacks (including 
PipelineStartEPCallbacks) for >O0 optimization pipeline. Yeah, then you need to 
add related passes to O0EPCallbacks (for BPF in this case).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89158

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to