[PATCH] D26431: [WebAssembly] Define __unix__ as part of the wasm target

Jacob Gravelle via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 8 16:36:28 PST 2016


jgravelle-google added a comment.

For the emscripten testsuite, specifically test_zlib relies on `#ifdef __unix__` to `#include <unistd.h>`, which it needs. The rest of the unistd tests don't actually care. So we could make that test pass by explicitly adding a define for __unix__ for it, but I figure it would be better to have things as similar as possible between wasm and asmjs.


https://reviews.llvm.org/D26431





More information about the cfe-commits mailing list