[clang] [clang][Sema] Add noinline check for __builtin_frame_address and __builtin_return_address (PR #82966)

[via cfe-commits]

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 47aee8b56d65e2bac5c7128424ff06134e454d83 
ef4c372a4d9364e6457c88a940c9af8666de0b74 -- clang/lib/Sema/SemaChecking.cpp 
clang/test/Sema/builtin-returnaddress.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 1861f7dace..cc8ba44f60 100644
--- a/clang/lib/Sema/SemaChecking.cpp
+++ b/clang/lib/Sema/SemaChecking.cpp
@@ -2736,8 +2736,8 @@ Sema::CheckBuiltinFunctionCall(FunctionDecl *FDecl, 
unsigned BuiltinID,
         TheCall->getArg(0)->EvaluateAsInt(Result, getASTContext())) {
       const char *BuiltinName =
           (BuiltinID == Builtin::BI__builtin_return_address)
-               ? "__builtin_return_address"
-               : "__builtin_frame_address";
+              ? "__builtin_return_address"
+              : "__builtin_frame_address";
 
       if (Result.Val.getInt() != 0) {
         Diag(TheCall->getBeginLoc(), diag::warn_frame_address)
@@ -2748,9 +2748,10 @@ Sema::CheckBuiltinFunctionCall(FunctionDecl *FDecl, 
unsigned BuiltinID,
       if (const FunctionDecl *FD = dyn_cast<FunctionDecl>(CurContext)) {
         if (!FD->hasAttr<NoInlineAttr>() && !FD->isMain()) {
           const char *ShortName =
-              (BuiltinID == Builtin::BI__builtin_return_address)
-                  ? "return" : "frame";
-          Diag(TheCall->getBeginLoc(), 
diag::warn_frame_address_missing_noinline)
+              (BuiltinID == Builtin::BI__builtin_return_address) ? "return"
+                                                                 : "frame";
+          Diag(TheCall->getBeginLoc(),
+               diag::warn_frame_address_missing_noinline)
               << BuiltinName << ShortName << TheCall->getSourceRange();
         }
       }

``````````

</details>


https://github.com/llvm/llvm-project/pull/82966
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits