[clang] [Sema] Fix c23 not checking CheckBoolLikeConversion (PR #79588)

via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 30 05:48:04 PST 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 e37a6008831b38477516362892dd3c0d97087eff aea883b73a072e8a7bd261a4d6b5885660bd215f -- clang/lib/Sema/SemaChecking.cpp clang/test/C/C2x/n3042.c clang/test/Sema/warn-int-in-bool-context.c
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/clang/lib/Sema/SemaChecking.cpp b/clang/lib/Sema/SemaChecking.cpp
index 6c31dc7245..86879e097e 100644
--- a/clang/lib/Sema/SemaChecking.cpp
+++ b/clang/lib/Sema/SemaChecking.cpp
@@ -16134,8 +16134,8 @@ static void CheckConditionalOperator(Sema &S, AbstractConditionalOperator *E,
 /// Check conversion of given expression to boolean.
 /// Input argument E is a logical expression.
 static void CheckBoolLikeConversion(Sema &S, Expr *E, SourceLocation CC) {
-  // While C23 does have bool as a keyword, we still need to run the bool-like 
-  // conversion checks as bools are still not used as the return type from 
+  // While C23 does have bool as a keyword, we still need to run the bool-like
+  // conversion checks as bools are still not used as the return type from
   // "boolean" operators or as the input type for conditional operator.
   if (S.getLangOpts().Bool && !S.getLangOpts().C23)
     return;

``````````

</details>


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


More information about the cfe-commits mailing list