From d0b54bb50e5110a004b41fc06dadf3fee70834b7 Mon Sep 17 00:00:00 2001 From: Kai Stierand Date: Tue, 25 Jul 2023 13:47:46 +0200 Subject: [PATCH] [Clang] use unsigned integer constants in unit-test | fixes build error on ppc64le-lld-multistage-test MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Fixes: /home/buildbots/ppc64le-lld-multistage-test/ppc64le-lld-multistage-test/llvm-project/third-party/unittest/googletest/include/gtest/gtest.h:1526:11: warning: comparison of integer expressions of different signedness: ‘const unsigned int’ and ‘const int’ [-Wsign-compare] /home/buildbots/ppc64le-lld-multistage-test/ppc64le-lld-multistage-test/llvm-project/third-party/unittest/googletest/include/gtest/gtest.h:1526:11: warning: comparison of integer expressions of different signedness: ‘const long unsigned int’ and ‘const int’ [-Wsign-compare] Reviewed By: cor3ntin Differential Revision: https://reviews.llvm.org/D156224 --- clang/unittests/libclang/LibclangTest.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/clang/unittests/libclang/LibclangTest.cpp b/clang/unittests/libclang/LibclangTest.cpp index b1f6534..295706c 100644 --- a/clang/unittests/libclang/LibclangTest.cpp +++ b/clang/unittests/libclang/LibclangTest.cpp @@ -1220,7 +1220,7 @@ static_assert(true, message); const char *Args[] = {"-xc++", "-std=c++26"}; ClangTU = clang_parseTranslationUnit(Index, fileName.c_str(), Args, std::size(Args), nullptr, 0, TUFlags); - ASSERT_EQ(clang_getNumDiagnostics(ClangTU), 0); + ASSERT_EQ(clang_getNumDiagnostics(ClangTU), 0u); std::optional staticAssertCsr; Traverse([&](CXCursor cursor, CXCursor parent) -> CXChildVisitResult { if (cursor.kind == CXCursor_StaticAssert) { @@ -1229,7 +1229,7 @@ static_assert(true, message); return CXChildVisit_Continue; }); ASSERT_TRUE(staticAssertCsr.has_value()); - size_t argCnt = 0; + int argCnt = 0; Traverse(*staticAssertCsr, [&argCnt](CXCursor cursor, CXCursor parent) { switch (argCnt) { case 0: -- 2.7.4