[PATCH] D36311: [ThinLTO] Add GraphTraits for FunctionSummaries
Easwaran Raman via llvm-commits
llvm-commits at lists.llvm.org
Sat Jan 20 14:37:02 PST 2018
I don't see D42311 as going in a different direction than this one. My
patch extends GraphTraits to iterate over the outgoing edges of node while
this patch specializes CallGraphTraits for ModuleSummaryIndex.
On Fri, Jan 19, 2018 at 8:43 PM, Teresa Johnson via Phabricator <
reviews at reviews.llvm.org> wrote:
> tejohnson added a comment.
>
> Sorry, I missed these being updated. Will take another pass through but it
> looks like you have addressed my comments.
>
> Easwaran - can this be used to do the call graph walks you need for
> synthetic counts propagation? If not we need to figure out why, because I
> see you are going in a different direction in https://reviews.llvm.org/
> D42311.
>
>
> https://reviews.llvm.org/D36311
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180120/22af0aa0/attachment.html>
More information about the llvm-commits
mailing list