[PATCH] D97722: [NewPM] Revamp pass names

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 2 11:55:14 PST 2021


arsenm added inline comments.


================
Comment at: llvm/include/llvm/Analysis/AliasAnalysis.h:1168
 
+  static StringRef name() { return "aa"; }
+
----------------
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?


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