aaron.ballman added a comment.

In http://reviews.llvm.org/D18073#399270, @ariccio wrote:

> In http://reviews.llvm.org/D18073#398882, @zaks.anna wrote:
>
> > "Since we are adding support for so many new APIs that are only available 
> > on Windows, could you please condition checking them only when we build for 
> > Windows. You probably can look and Language Options to figure that out."
> >
> > malloc-uses.c -> "alternative-malloc-api.c" ?
>
>
> Sorry for being so thickheaded about that the first go-around, I'm not 
> entirely sure what was up with me. I've changed the name of the file, but I 
> haven't yet figured out how to set up the Windows only tests to run on 
> Windows only. I'm assuming I can't simply:
>
>   #if defined(_WIN32)
>
>
> [...]
>
>   #endif
>
>
> ...because `lit` doesn't define `_WIN32` for me. Which Language Options 
> should I look in?


I believe that you can specify a triple using -triple i386-pc-win32 or 
something similar to force the compilation to be for a Windows target.


http://reviews.llvm.org/D18073



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to