[llvm] [StandardInstrumentations] Support -print-after-pass-number option (PR #87458)

Arthur Eubanks via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 9 22:51:02 PDT 2024


================
@@ -929,6 +942,11 @@ bool PrintIRInstrumentation::shouldPrintAfterPass(StringRef PassID) {
   return is_contained(printAfterPasses(), PassName);
 }
 
+bool PrintIRInstrumentation::shouldPrintAfterPassNumberCheck() {
----------------
aeubanks wrote:

`shouldPrintAfterPassNumberCheck`/`shouldPrintAfterPassNumber` is a bit confusing. perhaps `shouldPrintAfter/BeforePassNumber` should be `shouldPrintAfter/BeforeSomePassNumber`, and `shouldPrintAfter/BeforePassNumberCheck` should be `shouldPrintAfter/BeforeCurrentPassNumber`?

https://github.com/llvm/llvm-project/pull/87458


More information about the llvm-commits mailing list