[clang] [Clang] [Parser] Fixing all callers of `ParseExternalDeclaration` that didn't parse gnu attributes (PR #117148)

Alexey Bataev via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 25 10:54:34 PST 2024


================
@@ -2314,10 +2314,12 @@ Parser::DeclGroupPtrTy Parser::ParseOpenMPDeclarativeDirectiveWithExtDecl(
       // Here we expect to see some function declaration.
       if (AS == AS_none) {
         assert(TagType == DeclSpec::TST_unspecified);
-        ParsedAttributes EmptyDeclSpecAttrs(AttrFactory);
-        MaybeParseCXX11Attributes(Attrs);
+        ParsedAttributes DeclSpecAttrs(AttrFactory);
----------------
alexey-bataev wrote:

It is hard to tell now. `Attrs` comes from Parser::ParseExternalDeclaration. I don't remember the details already, but looks like we saw some missed attrs, if skipped this argument

https://github.com/llvm/llvm-project/pull/117148


More information about the cfe-commits mailing list