[PATCH] D81236: Improve LegacyPassManager API to correctly report modified status

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 18 23:24:00 PDT 2020


mehdi_amini added inline comments.


================
Comment at: llvm/include/llvm/Pass.h:216
+  AnalysisType &getAnalysisID(AnalysisID PI, Function &F,
+                              bool *Changed = nullptr);
 };
----------------
Can you document these added arguments? 
It isn't clear to me what are these about: I wouldn't expect an analysis to change the IR for example.


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

https://reviews.llvm.org/D81236





More information about the llvm-commits mailing list