[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