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

[Charles Saternos via llvm-commits]

OK, I've been looking into the issue. Not sure why its happening since the
test passes when I build locally. I'll keep investigating.

Thanks,
Charles

On Sat, Feb 17, 2018 at 7:35 PM, Simon Pilgrim <llvm-dev at redking.me.uk>
wrote:

> Charles, I'm sorry but I had to revert this at rL325454 to try and fix the
> buildbot failures. Simon.
> On 17/02/2018 22:24, Charles Saternos via llvm-commits wrote:
>
> 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
>>
>
>
>
> _______________________________________________
> llvm-commits mailing listllvm-commits at lists.llvm.orghttp://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180217/d4424c7f/attachment.html>