From 0ae342f45bedd29e34690de087011a9da4db6a65 Mon Sep 17 00:00:00 2001 From: Jie Fu Date: Fri, 14 Apr 2023 09:47:21 +0800 Subject: [PATCH] [lldb][test] Fix -Wsign-compare in RegisterFlagsTest.cpp (NFC) /data/llvm-project/third-party/unittest/googletest/include/gtest/gtest.h:1526:11: error: comparison of integers of different signs: 'const unsigned long long' and 'const int' [-Werror,-Wsign-compare] if (lhs == rhs) { ~~~ ^ ~~~ /data/llvm-project/third-party/unittest/googletest/include/gtest/gtest.h:1553:12: note: in instantiation of function template specialization 'testing::internal::CmpHelperEQ' requested here return CmpHelperEQ(lhs_expression, rhs_expression, lhs, rhs); ^ /data/llvm-project/lldb/unittests/Target/RegisterFlagsTest.cpp:128:3: note: in instantiation of function template specialization 'testing::internal::EqHelper::Compare' requested here ASSERT_EQ(0x12345678ULL, rf.ReverseFieldOrder(0x12345678)); ^ /data/llvm-project/third-party/unittest/googletest/include/gtest/gtest.h:2056:32: note: expanded from macro 'ASSERT_EQ' ^ /data/llvm-project/third-party/unittest/googletest/include/gtest/gtest.h:2040:54: note: expanded from macro 'GTEST_ASSERT_EQ' ASSERT_PRED_FORMAT2(::testing::internal::EqHelper::Compare, val1, val2) ^ 1 error generated. --- lldb/unittests/Target/RegisterFlagsTest.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lldb/unittests/Target/RegisterFlagsTest.cpp b/lldb/unittests/Target/RegisterFlagsTest.cpp index 3819c6f..c24a6d9 100644 --- a/lldb/unittests/Target/RegisterFlagsTest.cpp +++ b/lldb/unittests/Target/RegisterFlagsTest.cpp @@ -125,11 +125,11 @@ TEST(RegisterFlagsTest, RegisterFlagsPadding) { TEST(RegisterFieldsTest, ReverseFieldOrder) { // Unchanged RegisterFlags rf("", 4, {make_field(0, 31)}); - ASSERT_EQ(0x12345678ULL, rf.ReverseFieldOrder(0x12345678)); + ASSERT_EQ(0x12345678ULL, (unsigned long long)rf.ReverseFieldOrder(0x12345678)); // Swap the two halves around. RegisterFlags rf2("", 4, {make_field(16, 31), make_field(0, 15)}); - ASSERT_EQ(0x56781234ULL, rf2.ReverseFieldOrder(0x12345678)); + ASSERT_EQ(0x56781234ULL, (unsigned long long)rf2.ReverseFieldOrder(0x12345678)); // Many small fields. RegisterFlags rf3("", 4, -- 2.7.4