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

[Kuter Dinel via Phabricator via llvm-commits]

kuter added a comment.

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.
I think most of the state change should happen in the `::run` function.



================
Comment at: llvm/lib/Transforms/IPO/Attributor.cpp:1150
   }
   return ManifestChange;
 }
----------------
The state is stuck as `MANIFEST`


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

https://reviews.llvm.org/D86678