[PATCH] [ms-cxxabi] Move CodeGenVTables::needsVTTParameter to ItaniumCXXABI.

Reid Kleckner rnk at google.com
Thu Jun 27 09:47:42 PDT 2013



================
Comment at: test/CodeGenCXX/microsoft-abi-virtual-inheritance.cpp:11
@@ +10,2 @@
+
+C::C() {}
----------------
You could check the mangling here to assert that we don't add the hidden parameter, right?

================
Comment at: test/CodeGenCXX/microsoft-abi-virtual-inheritance.cpp:1
@@ +1,2 @@
+// RUN: %clang_cc1 %s -fno-rtti -cxx-abi microsoft -triple=i386-pc-win32 -emit-llvm -o %t
+
----------------
I'd roll this into the microsoft-abi-structors.cpp test case


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



More information about the cfe-commits mailing list