[PATCH] D18073: Add memory allocating functions

Alexander Riccio via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 12 17:01:41 PDT 2016


ariccio added a comment.

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?


http://reviews.llvm.org/D18073





More information about the cfe-commits mailing list