[llvm] r223802 - IR: Split Metadata from Value

Duncan P. N. Exon Smith dexonsmith at apple.com
Tue Dec 9 12:29:54 PST 2014


> On 2014 Dec 9, at 12:22, David Blaikie <dblaikie at gmail.com> wrote:
> 
> 
> 
> On Tue, Dec 9, 2014 at 12:07 PM, Hans Wennborg <hans at chromium.org> wrote:
> This broke the MSVC build, see e.g. [1]. I suspect it is because MSVC
> (except recent versions) doesn't support LLVM_EXPORTED.
> 
> I take it you meant LLVM_EXPLICIT?
> 
> 
>  
> I tried
> removing those operator bool()'s, but there were more errors.
> 
> Does your patch compile if you remove LLVM_EXPORTED? I suspect that
> might reveal the same problem on Mac/Linux.
> 
>  - Hans
> 
>  1. http://build.chromium.org/p/chromium.fyi/builders/CrWinClang/builds/660/steps/runhooks/logs/stdio
> 
> Somewhat surprised that compiles anywhere, really... Curious.

Yeah, IMO it should have failed to compile in clang too.





More information about the llvm-commits mailing list