[llvm] r325448 - [ThinLTO] Add GraphTraits for FunctionSummaries

[Charles Saternos via llvm-commits]

Hey Davide,

It's been reported here: https://bugs.llvm.org/show_bug.cgi?id=32556

The use of std::function rather than function_ref is related to some
problem in clang.

Thanks,
Charles

On Sat, Feb 17, 2018 at 5:04 PM, Davide Italiano <davide at freebsd.org> wrote:

> On Sat, Feb 17, 2018 at 1:39 PM, Charles Saternos via llvm-commits
> <llvm-commits at lists.llvm.org> wrote:
> > Author: ncharlie
> > Date: Sat Feb 17 13:39:24 2018
> > New Revision: 325448
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=325448&view=rev
> > Log:
> > [ThinLTO] Add GraphTraits for FunctionSummaries
> >
> > Add GraphTraits definitions to the FunctionSummary and
> ModuleSummaryIndex classes. These GraphTraits will be used to construct
> find SCC's in ThinLTO analysis passes.
> >
> > Second attempt, since last patch caused stage2 build to fail (now using
> function_ref rather than std::function).
> >
>
> Charlie, what's the bug in clang? It would be good to fix it, regardless.
>
> Thanks,
>
> --
> Davide
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180217/ec8e489d/attachment.html>