[PATCH] clang-cl: Use MS C++ ABI
Hans Wennborg
hans at chromium.org
Thu Aug 8 12:57:14 PDT 2013
================
Comment at: lib/Driver/Tools.cpp:3728
@@ +3727,3 @@
+
+ CmdArgs.push_back("-cxx-abi");
+ CmdArgs.push_back("microsoft");
----------------
Reid Kleckner wrote:
> This should have a FIXME: Make this the default for the win32 triple.
>
> Right now i686-pc-win32 + -cxx-abi itanium is a weird mismash of the C calling conventions plus the Itanium C++ runtime stuff that we should really eliminate.
Done.
================
Comment at: test/Driver/cl-abi.c:1
@@ +1,2 @@
+// Don't attempt slash switches on msys bash.
+// REQUIRES: shell-preserves-root
----------------
Reid Kleckner wrote:
> Let's try to have fewer test files. I've read on the lists that more files basically makes lit slower.
OK. I've moved this test into test/Driver/cl.c instead.
http://llvm-reviews.chandlerc.com/D1329
COMMIT
http://llvm-reviews.chandlerc.com/rL188000
More information about the cfe-commits
mailing list