[PATCH] D82270: Added a Printer to the FunctionPropertiesAnalysis

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 23 13:59:20 PDT 2020


jdoerfert added a comment.

In D82270#2106860 <https://reviews.llvm.org/D82270#2106860>, @mtrofin wrote:

> What's the value of having the result type have the analyze method? Is there a scenario where we want to get the result and then re-analyze it?


In the new-PM if you preserve the analysis it doesn't need to be recomputed, I thought. Or maybe I misunderstand your question. Could you elaborate?

[EDIT: Do you mean the `...Info` struct? It is used by two passes, right?]


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

https://reviews.llvm.org/D82270





More information about the llvm-commits mailing list