This revision was automatically updated to reflect the committed changes.
Closed by commit rL320925: [libcxx] Add WebAssembly support (authored by sbc,
committed by ).
Changed prior to commit:
https://reviews.llvm.org/D41073?vs=127242&id=127251#toc
Repository:
rL LLVM
https://reviews.llvm.o
ncw updated this revision to Diff 127242.
ncw added a comment.
Herald added a subscriber: krytarowski.
Although Dan has already reviewed, I've sneakily added in another change here
to help compilation against Musl.
Musl is Linux-only, or rather, provides the linux ABI. For Wasm, `__linux`
isn't
sunfish added a comment.
Either way. I won't be able to get to it until next week, so feel free to land
it earlier.
Repository:
rCXX libc++
https://reviews.llvm.org/D41073
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm
sbc100 added a comment.
Do you want to land this dan? Or I can if you prefer.
Repository:
rCXX libc++
https://reviews.llvm.org/D41073
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-comm
sunfish accepted this revision.
sunfish added a comment.
This revision is now accepted and ready to land.
Yes, wasm has its own object format, so this change makes sense.
Repository:
rCXX libc++
https://reviews.llvm.org/D41073
___
cfe-commits mai
ncw created this revision.
Herald added subscribers: cfe-commits, aheejin, jfb.
Herald added a reviewer: EricWF.
It turns out that this is the only change required in libcxx for it to compile
with the new `wasm32-unknown-unknown-wasm` target recently added to Clang.
I haven't done much testing o
