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

[Derek Schuff via cfe-commits]

dschuff added a comment.

Yeah, I don't mean for this to be the final word; mostly this just matches asm.js.
I do agree with you that the unixy stuff goes with the emscripten environment rather than wasm per se. I had thought we might define emscripten as an "OS" (i.e. the third or maybe even 4th part of the triple) in LLVM, because currently it affects not just defines but also things like emscripten exception handling (which we currently use a backend flag for).


https://reviews.llvm.org/D26431