[clang] [clang] Update argument checking tablegen code to use a 'full' name (PR #99993)
via cfe-commits
cfe-commits at lists.llvm.org
Mon Jul 22 15:55:44 PDT 2024
github-actions[bot] wrote:
<!--LLVM CODE FORMAT COMMENT: {clang-format}-->
:warning: C/C++ code formatter, clang-format found issues in your code. :warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
``````````bash
git-clang-format --diff 497ea1d84951626dea5bf644fef2d99e145e21ac 5f8b68d9dfd7680adeadbae1d761dc03a2455685 --extensions h,cpp -- clang/include/clang/Basic/AttributeCommonInfo.h clang/lib/Basic/Attributes.cpp clang/lib/Parse/ParseDecl.cpp clang/lib/Sema/SemaStmtAttr.cpp clang/utils/TableGen/ClangAttrEmitter.cpp
``````````
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/clang/lib/Parse/ParseDecl.cpp b/clang/lib/Parse/ParseDecl.cpp
index 28c69f9981..39c590a57e 100644
--- a/clang/lib/Parse/ParseDecl.cpp
+++ b/clang/lib/Parse/ParseDecl.cpp
@@ -322,7 +322,7 @@ static bool attributeHasIdentifierArg(const IdentifierInfo &II,
#define CLANG_ATTR_IDENTIFIER_ARG_LIST
return llvm::StringSwitch<bool>(FullName)
#include "clang/Parse/AttrParserStringSwitches.inc"
- .Default(false);
+ .Default(false);
#undef CLANG_ATTR_IDENTIFIER_ARG_LIST
}
@@ -349,7 +349,7 @@ static bool attributeHasVariadicIdentifierArg(const IdentifierInfo &II,
#define CLANG_ATTR_VARIADIC_IDENTIFIER_ARG_LIST
return llvm::StringSwitch<bool>(FullName)
#include "clang/Parse/AttrParserStringSwitches.inc"
- .Default(false);
+ .Default(false);
#undef CLANG_ATTR_VARIADIC_IDENTIFIER_ARG_LIST
}
@@ -362,7 +362,7 @@ static bool attributeTreatsKeywordThisAsIdentifier(const IdentifierInfo &II,
#define CLANG_ATTR_THIS_ISA_IDENTIFIER_ARG_LIST
return llvm::StringSwitch<bool>(FullName)
#include "clang/Parse/AttrParserStringSwitches.inc"
- .Default(false);
+ .Default(false);
#undef CLANG_ATTR_THIS_ISA_IDENTIFIER_ARG_LIST
}
@@ -388,7 +388,7 @@ static bool attributeIsTypeArgAttr(const IdentifierInfo &II,
#define CLANG_ATTR_TYPE_ARG_LIST
return llvm::StringSwitch<bool>(FullName)
#include "clang/Parse/AttrParserStringSwitches.inc"
- .Default(false);
+ .Default(false);
#undef CLANG_ATTR_TYPE_ARG_LIST
}
@@ -431,7 +431,7 @@ static bool attributeParsedArgsUnevaluated(const IdentifierInfo &II,
#define CLANG_ATTR_ARG_CONTEXT_LIST
return llvm::StringSwitch<bool>(FullName)
#include "clang/Parse/AttrParserStringSwitches.inc"
- .Default(false);
+ .Default(false);
#undef CLANG_ATTR_ARG_CONTEXT_LIST
}
diff --git a/clang/utils/TableGen/ClangAttrEmitter.cpp b/clang/utils/TableGen/ClangAttrEmitter.cpp
index d9fb16f5f0..f504b1de14 100644
--- a/clang/utils/TableGen/ClangAttrEmitter.cpp
+++ b/clang/utils/TableGen/ClangAttrEmitter.cpp
@@ -2372,11 +2372,10 @@ void PragmaClangAttributeSupport::generateParsingHelpers(raw_ostream &OS) {
OS << "}\n\n";
}
-template <typename Fn>
-static void forEachSpelling(const Record &Attr, Fn &&F) {
+template <typename Fn> static void forEachSpelling(const Record &Attr, Fn &&F) {
std::vector<FlattenedSpelling> Spellings = GetFlattenedSpellings(Attr);
for (const FlattenedSpelling &S : Spellings) {
- F(S);
+ F(S);
}
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/99993
More information about the cfe-commits
mailing list