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

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 17 16:35:43 PST 2018


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 
> <mailto: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 <mailto: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
>     <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 list
> llvm-commits at lists.llvm.org
> http://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/20180218/c929c981/attachment.html>


More information about the llvm-commits mailing list