patcheng added a comment.
Thanks @sbc100.
BTW I don't have commit access. When it's ready to be committed, I would need
help with that.
Thanks
https://reviews.llvm.org/D41937
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.ll
patcheng updated this revision to Diff 129572.
patcheng edited the summary of this revision.
patcheng added a comment.
Added -lc++abi per @sbc100 's suggestion
https://reviews.llvm.org/D41937
Files:
lib/Driver/ToolChains/WebAssembly.cpp
lib/Driver/ToolChains/WebAssembly.h
test/Driver/wasm
patcheng added inline comments.
Comment at: lib/Driver/ToolChains/WebAssembly.cpp:151
+ case ToolChain::CST_Libcxx:
+CmdArgs.push_back("-lc++");
+break;
sbc100 wrote:
> Can you add -lc++abi too? Then I can drop D41966
Sure, but I am not in front of a com
patcheng created this revision.
Herald added subscribers: cfe-commits, sunfish, aheejin, jgravelle-google,
sbc100, dschuff, jfb.
Herald added a reviewer: EricWF.
Referenced implementation from Fuchsia and Darwin Toolchain.
Still only support CST_Libcxx.
Now checks that the argument is really '-
