[PATCH] D28381: [WebAssembly] Always inline atomics

Dan Gohman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 6 08:16:12 PST 2017


sunfish added a comment.

Since we may soon have people prototyping actual atomics, I'd prefer to find a less invasive way to fix this.

Would it be difficult to enable atomic.c in Emscripten's compiler-rt build, to define these libcalls (assuming that the problem is just that they're not currently defined)?

Alternatively, could we make the change in the patch guarded by `if (CodeGenOpts.ThreadModel == "single")` or so, so that it doesn't affect the "posix" case?


https://reviews.llvm.org/D28381





More information about the cfe-commits mailing list