[PATCH] D14348: Add Printable class to simplify the construction of Print helpers.

[David Blaikie via llvm-commits]

On Thu, Dec 3, 2015 at 3:29 PM, Matthias Braun via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> MatzeB added a comment.
>
> I had to revert this patch because two gcc buildbots and all the msvc bots
> choke on ambiguous operator<< overloads. Examples:
>
>
> http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-autoconf/builds/12622/steps/build%20fresh%20clang/logs/stdio
>
> http://lab.llvm.org:8011/builders/polly-amd64-linux/builds/36852/steps/build_polly/logs/stdio
>
> http://lab.llvm.org:8011/builders/lld-x86_64-win7/builds/27400/steps/build_Lld/logs/stdio
>
> My current guess is that std::function implicitely converts to void* or
> void* implicitely converts to std::function on these systems. So I guess to
> get this working I have to go back to the shim class after all :-(
>

Fair guess - though if you've got access to such a revision of GCC such
that you could test/demonstrate this, it'd be good. I'm always hesitant to
workaround something without knowing exactly what it is that's being worked
around. (especially so we can leave a comment to rip out teh workaround
when we drop support for whatever version of the compiler/standard library
has this bug).

- Dave


>
>
> Repository:
>   rL LLVM
>
> http://reviews.llvm.org/D14348
>
>
>
> _______________________________________________
> 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/20151203/ca53a91c/attachment.html>