From: Francois Pichet Date: Mon, 31 Jan 2011 04:54:32 +0000 (+0000) Subject: Allow Microsoft attributes in a constructor's parameter list. X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=dfaa5fb15da115315513ce356169450a5963977f;p=clang Allow Microsoft attributes in a constructor's parameter list. This fixes a few compile errors when parsing from MSVC 2008 with clang. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@124573 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Parse/ParseDecl.cpp b/lib/Parse/ParseDecl.cpp index c4fce66b65..24e41d0737 100644 --- a/lib/Parse/ParseDecl.cpp +++ b/lib/Parse/ParseDecl.cpp @@ -2512,6 +2512,10 @@ bool Parser::isConstructorDeclarator() { if (SS.isSet() && Actions.ShouldEnterDeclaratorScope(getCurScope(), SS)) DeclScopeObj.EnterDeclaratorScope(); + // Optionally skip Microsoft attributes. + ParsedAttributes Attrs; + MaybeParseMicrosoftAttributes(Attrs); + // Check whether the next token(s) are part of a declaration // specifier, in which case we have the start of a parameter and, // therefore, we know that this is a constructor. diff --git a/test/Parser/MicrosoftExtensions.cpp b/test/Parser/MicrosoftExtensions.cpp index 1d5811496a..fd0d7d50d0 100644 --- a/test/Parser/MicrosoftExtensions.cpp +++ b/test/Parser/MicrosoftExtensions.cpp @@ -22,6 +22,14 @@ extern "C" { int foo5([SA_Post(attr=1)] void *param); } +class class_attr { +public: + class_attr([SA_Pre(Null=SA_No,NullTerminated=SA_Yes)] int a) + { + } +}; + + void uuidof_test1() {