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

[Chen, Yuanfang via llvm-commits]

Indeed. Will do.

________________________________________
From: Alina Sbirlea via Phabricator <reviews at reviews.llvm.org>
Sent: Tuesday, July 28, 2020 10:48 AM
To: Chen, Yuanfang; asbirlea at google.com; aeubanks at google.com; hans at chromium.org
Cc: llvm-commits at lists.llvm.org; 88888yl at gmail.com; czhengsz at cn.ibm.com
Subject: [PATCH] D84772: [NewPM][PassInstrument] Add a new kind of before-pass callback that only get called if the pass is not skipped

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);
----------------
Perhaps `BeforePassRequiredFunc` (or similar) is more informative that these callbacks are run for required passes?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84772