[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