[PATCH] D18073: Add memory allocating functions

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 14 09:20:17 PDT 2016


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





More information about the cfe-commits mailing list