[llvm] [AMDGPU] Fix unittest sign-compare (PR #155235)
via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 25 05:00:45 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-backend-amdgpu
Author: Robert Imschweiler (ro-i)
<details>
<summary>Changes</summary>
Apparently, some buildbots build unittests with `-Werror,-Wsign-compare`: https://lab.llvm.org/buildbot/#/builders/145/builds/9299
---
Full diff: https://github.com/llvm/llvm-project/pull/155235.diff
1 Files Affected:
- (modified) llvm/unittests/Target/AMDGPU/AMDGPUUnitTests.cpp (+1-1)
``````````diff
diff --git a/llvm/unittests/Target/AMDGPU/AMDGPUUnitTests.cpp b/llvm/unittests/Target/AMDGPU/AMDGPUUnitTests.cpp
index d01d808f988ba..593991c71d706 100644
--- a/llvm/unittests/Target/AMDGPU/AMDGPUUnitTests.cpp
+++ b/llvm/unittests/Target/AMDGPU/AMDGPUUnitTests.cpp
@@ -335,7 +335,7 @@ TEST(AMDGPU, TestGetNamedOperandIdx) {
if (OpName == AMDGPU::OpName::NUM_OPERAND_NAMES)
continue;
int16_t RetrievedIdx = AMDGPU::getNamedOperandIdx(Opcode, OpName);
- EXPECT_EQ(Idx, RetrievedIdx)
+ EXPECT_EQ(Idx, static_cast<unsigned>(RetrievedIdx))
<< "Opcode " << Opcode << " (" << MCII->getName(Opcode) << ')';
}
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/155235
More information about the llvm-commits
mailing list