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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to