[PATCH] D60812: Don't error on CMAKE_SYSTEM_NAME=Wasi

[Matthew Bauer via Phabricator via llvm-commits]

matthewbauer marked an inline comment as done.
matthewbauer added inline comments.


================
Comment at: llvm/cmake/modules/HandleLLVMOptions.cmake:129
     endif()
   else(FUCHSIA OR UNIX)
+    if(${CMAKE_SYSTEM_NAME} MATCHES "Wasi")
----------------
sunfish wrote:
> I don't know a lot about CMake, but would be better to add a predicate for `WASI` so that we can change these `FUCHSIA OR UNIX` conditions into `FUCHSIA OR UNIX OR WASI` conditions?
Maybe, although we'd have to make sure that LLVM_ON_UNIX and LLVM_HAVE_LINK_VERSION_SCRIPT are okay to set above. I think LLVM_HAVE_LINK_VERSION_SCRIPT is fine (lld should allow it) but not sure about LLVM_ON_UNIX.

We aren't really using this right now although at some point someone could port clang and llvm to run on Wasi I suppose. In the libcxxabi case LLVM_ON_UNIX might not be needed at all. We aren't actually building LLVM for Wasm, just using it for some headers.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60812/new/

https://reviews.llvm.org/D60812