[PATCH] D86678: [Attributor] Add a phase flag to Attributor

Shinji Okumura via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 27 00:51:57 PDT 2020


okura added a comment.

In D86678#2241129 <https://reviews.llvm.org/D86678#2241129>, @kuter wrote:

> Thank you for creating this. I was thinking of something like this rather than having multiple flags.
> In D86587 <https://reviews.llvm.org/D86587> I iterate on functions multiple times and I delete AAs early after manifesting them.
> (Manifest happens multiple times).
> I think most of the state change should happen in the `::run` function.

OK, I'll move state changes into `::run`


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

https://reviews.llvm.org/D86678



More information about the llvm-commits mailing list