[cfe-commits] [PATCH][MS] - type_info forward class declaration

Douglas Gregor dgregor at apple.com
Thu Aug 26 07:28:01 PDT 2010


On Aug 26, 2010, at 3:17 AM, Francois Pichet wrote:

> Hi,
> 
> Here is my patch of the day. This one is small and simple.
> clang will predefine a forward class declaration for type_info in
> Microsoft mode.
> This is required to be MSVC compatible.

This test case is clearly broken:

Index: test/Lexer/ms-extensions.c
===================================================================
--- test/Lexer/ms-extensions.c	(revision 112178)
+++ test/Lexer/ms-extensions.c	(working copy)
@@ -23,3 +23,6 @@
         unsigned short s = USHORT;
         unsigned char c = UCHAR;
 }
+
+// type_info forward class declaration is built-in.
+voif f(type_info &a);


"void" is misspelled, and type_info doesn't (and shouldn't) exist in C. Please provide a proper test case with an updated patch.

Does Visual C++ also implicitly declare namespace "std" or "std::type_info"? That's the kind of thing you'll also want to check.

	- Doug





More information about the cfe-commits mailing list