[PATCH] D82521: Refactor FunctionPropertiesAnalysis

Mircea Trofin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 25 08:34:23 PDT 2020


mtrofin added inline comments.


================
Comment at: llvm/include/llvm/Analysis/ML/FunctionPropertiesAnalysis.h:44
+
+  void analyze(const Function &F);
+};
----------------
I think (D82270) we were saying we would have a static factory method in FunctionPropertiesInfo, instead of a member. We'd then (maybe later CL) try to treat a FunctionPropertiesInfo instance as immutable. 


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

https://reviews.llvm.org/D82521





More information about the llvm-commits mailing list