[PATCH] D43034: Use `wasm` rather than `wasm32` as the as the default WebAssembly arch name

Dan Gohman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 7 16:20:13 PST 2018


sunfish added a comment.

As an example, you can have code like this:

  #ifdef __wasm__
    // do stuff with __builtin_wasm_mem_grow and __builtin_wasm_mem_size
  #endif

which works for both wasm32 and wasm64.

> On place I particularly want to avoid the 32 is in the tool names: wasm32-unknown-unknown-clang is bad enough, wasm-unknown-unknown-clang better.

I also hear people complain about the "unknown-unknown" part...

> Ideally just wasm-clang.

Is there something using "wasm32-clang" today?


Repository:
  rL LLVM

https://reviews.llvm.org/D43034





More information about the llvm-commits mailing list