[llvm] [X86][ABIVerifier] Verify floating point ABI correctness on 64-bit target (PR #111690)

via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 9 07:48:08 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 0cdb978906b060508204788a8cfcb5c7d9155e32 c886cc498da6eabf72bcfbce041bfc244abb61b3 --extensions cpp -- llvm/lib/IR/Verifier.cpp
``````````

</details>

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

``````````diff
diff --git a/llvm/lib/IR/Verifier.cpp b/llvm/lib/IR/Verifier.cpp
index eff0fde17e..ddd1ce2fc1 100644
--- a/llvm/lib/IR/Verifier.cpp
+++ b/llvm/lib/IR/Verifier.cpp
@@ -2096,12 +2096,12 @@ void Verifier::verifyX86ABI(FunctionType *FT, AttributeList Attrs,
   StringRef TF = Attrs.getFnAttr("target-features").getValueAsString();
   // Check SSE feature.
   Check(!TT.isArch64Bit() || !TF.contains("-sse,") ||
-        !FT->getReturnType()->isFloatTy(),
+            !FT->getReturnType()->isFloatTy(),
         "SSE register return with SSE disabled", V);
   // Check SSE2 feature.
   Check(!TT.isArch64Bit() || !TF.contains("-sse2") ||
-        (!FT->getReturnType()->isDoubleTy() &&
-         !FT->getReturnType()->is16bitFPTy()),
+            (!FT->getReturnType()->isDoubleTy() &&
+             !FT->getReturnType()->is16bitFPTy()),
         "SSE2 register return with SSE2 disabled", V);
   // Check EVEX512 feature.
   if (MaxParameterWidth >= 512)

``````````

</details>


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


More information about the llvm-commits mailing list