[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