[PATCH] D36233: [ThinLTO] Add FunctionAttrs to ThinLTO index

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 4 07:51:18 PDT 2017


tejohnson added a comment.

In https://reviews.llvm.org/D36233#831907, @ncharlie wrote:

> In https://reviews.llvm.org/D36233#831895, @tejohnson wrote:
>
> > Do you have commit access and if not do you want me to submit for you?
>
>
> Yup, I got commit access last week.


Great!

> 
> 
>> Do you have follow-on patches that use this about ready to go for review?
> 
> I'm currently working on cleaning up the patch for the GraphTraits for the function summary (planning on submitting that today), then I'll be able to add ReadNone/ReadOnly/NoRecurse propagation (since they all require SCC's). Do you want me to hold off on submitting this patch until one of those are ready to go?

Nah, go ahead, let's get this one in.
Thanks!


https://reviews.llvm.org/D36233





More information about the llvm-commits mailing list