[llvm] [AArch64] Avoid warning about comparison of different signedness. NFC. (PR #159337)
via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 17 04:52:13 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-backend-aarch64
Author: Martin Storsjö (mstorsjo)
<details>
<summary>Changes</summary>
This fixes the following warning when compiled with GCC:
../lib/Target/AArch64/AArch64ISelLowering.cpp: In function ‘bool shouldLowerTailCallStackArg(const llvm::MachineFunction&, const llvm::CCValAssign&, llvm::SDValue, llvm::ISD::ArgFlagsTy, int)’:
../lib/Target/AArch64/AArch64ISelLowering.cpp:9310: warning: comparison of integer expressions of different signedness: ‘uint64_t’ {aka ‘long unsigned int’} and ‘int64_t’ {aka ‘long int’} [-Wsign-compare]
9310 | if (SizeInBits / 8 != MFI.getObjectSize(FI))
|
---
Full diff: https://github.com/llvm/llvm-project/pull/159337.diff
1 Files Affected:
- (modified) llvm/lib/Target/AArch64/AArch64ISelLowering.cpp (+1-1)
``````````diff
diff --git a/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp b/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
index f6389aad96bf8..4af0d2f089c23 100644
--- a/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
+++ b/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
@@ -9307,7 +9307,7 @@ static bool shouldLowerTailCallStackArg(const MachineFunction &MF,
if (CallOffset != MFI.getObjectOffset(FI))
return true;
uint64_t SizeInBits = LoadNode->getMemoryVT().getFixedSizeInBits();
- if (SizeInBits / 8 != MFI.getObjectSize(FI))
+ if (SizeInBits / 8 != static_cast<uint64_t>(MFI.getObjectSize(FI)))
return true;
return false;
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/159337
More information about the llvm-commits
mailing list