[PATCH] D50277: [WebAssembly] Support for atomic fences

Dan Gohman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 24 17:26:03 PDT 2019


sunfish added a comment.

If I follow the discussion correctly, there are subtle ABI issues either way, so making fence a RMW now doesn't give us fully a future-proof ABI anyway. Consequently, I would prefer to make this patch conditional on the Emscripten triple, as wasm32-wasi and wasm32-unknown-unknown don't require Emscripten compatibility.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D50277





More information about the llvm-commits mailing list