================
@@ -3035,11 +3035,14 @@ Parser::DeclGroupPtrTy 
Parser::ParseCXXClassMemberDeclaration(
   }
 
   ParsedAttributes DeclSpecAttrs(AttrFactory);
-  MaybeParseMicrosoftAttributes(DeclSpecAttrs);
-
   // Hold late-parsed attributes so we can attach a Decl to them later.
   LateParsedAttrList CommonLateParsedAttrs;
 
+  while (MaybeParseCXX11Attributes(DeclAttrs) ||
----------------
AaronBallman wrote:

I actually think the `MaybeParseCXX11Attributes()` on line 3007 may be 
incorrect (a pre-existing bug), because we seem to quietly accept: 
https://godbolt.org/z/s9qbd8jqn but the grammar for member-declaration does not 
allow an attribute list to appear there: 
https://eel.is/c++draft/class#nt:member-declaration

https://github.com/llvm/llvm-project/pull/133107
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to