[PATCH] Use the MS ABI for Win32 target by default

[Reid Kleckner]

  I think all the test/CodeGen* changes are good, and should probably land now.

  The test/CXX and test/SemaCXX changes should probably be reviewed separately by Richard.


================
Comment at: test/CXX/drs/dr2xx.cpp:543
@@ -542,3 +542,3 @@
   struct C : A, B {
-    virtual ~C();
+    virtual ~C(); // expected-error 0-1 {{'operator delete' found in multiple base classes}}
   };
----------------
So, I remember we decided to do it this way, but now we've essentially weakened this test to pass if we fail to diagnose this in the Itanium C++ ABI.  Can you remind me what the issue is here?  This has to do with differing operator delete lookup behavior?


http://llvm-reviews.chandlerc.com/D2401