[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