[PATCH] D97722: [NewPM] Revamp pass names

Yuanfang Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 2 11:59:38 PST 2021


ychen added inline comments.


================
Comment at: llvm/include/llvm/Analysis/AliasAnalysis.h:1168
 
+  static StringRef name() { return "aa"; }
+
----------------
arsenm wrote:
> We already have multiple places where pass names are specified. We have getPassName, which many passes unnecessarily overload on top of the pass name set in the macros. Where does this version fit in this?
`getPassName` is for legacy PM. This is the new PM counterpart.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97722



More information about the llvm-commits mailing list