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

[Matthias Braun via llvm-commits]

Unfortunately I don't, maybe some of the MSVC developers could take a look?
CCing some "random" people that may have msvc handy.

- Matthias

> On Dec 3, 2015, at 3:32 PM, David Blaikie via llvm-commits <llvm-commits at lists.llvm.org> wrote:
> 
> 
> 
> 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
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits