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

Thu Aug 27 00:52:45 PDT 2020

kuter added a comment.

In D86678#2241217 <https://reviews.llvm.org/D86678#2241217>, @okura wrote:

> 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`

Thank you.

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

https://reviews.llvm.org/D86678