[clang] [Clang] Fix incorrect handling of #pragma {GCC} unroll N in dependent context (PR #90240)

via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 26 10:55:00 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 bf67610a8ac07ec143f243f129eecd456aa63383 131783211a23d007b5b6f1d5691306df4dec716b -- clang/lib/Sema/SemaStmtAttr.cpp clang/test/Parser/pragma-unroll.cpp
``````````

</details>

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

``````````diff
diff --git a/clang/lib/Sema/SemaStmtAttr.cpp b/clang/lib/Sema/SemaStmtAttr.cpp
index 4dfdfd7aec..d3e76f2829 100644
--- a/clang/lib/Sema/SemaStmtAttr.cpp
+++ b/clang/lib/Sema/SemaStmtAttr.cpp
@@ -113,8 +113,9 @@ static Attr *handleLoopHintAttr(Sema &S, Stmt *St, const ParsedAttr &A,
       if (!ValueExpr->isValueDependent()) {
         llvm::APSInt ValueAPS;
         ExprResult R = S.VerifyIntegerConstantExpression(ValueExpr, &ValueAPS);
-        assert(!R.isInvalid() && "unroll count value must be a valid value, it's "
-                                "should be checked in Sema::CheckLoopHintExpr");
+        assert(!R.isInvalid() &&
+               "unroll count value must be a valid value, it's "
+               "should be checked in Sema::CheckLoopHintExpr");
         (void)R;
         // The values of 0 and 1 block any unrolling of the loop.
         if (ValueAPS.isZero() || ValueAPS.isOne())

``````````

</details>


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


More information about the cfe-commits mailing list