Merge pull request ceph#65012 from aainscow/clang_signed_compare

src: Add sign-compare warnings to clang

Reviewed-by: Kefu Chai <[email protected]>

Author: aainscow

src/CMakeLists.txt

@@ -113,6 +113,10 @@ if(NOT MSVC)
   add_compile_options($<$<COMPILE_LANGUAGE:CXX>:-Wno-ignored-qualifiers>)
 endif()
 
+if (CMAKE_CXX_COMPILER_ID STREQUAL Clang)
+  add_compile_options($<$<COMPILE_LANGUAGE:CXX>:-Wsign-compare>)
+endif()
+
 add_compile_options($<$<COMPILE_LANGUAGE:CXX>:-ftemplate-depth-1024>)
 
 # Because Boost can't be bothered to not include its own deprecated headers

src/test/osd/TestECBackend.cc

@@ -158,12 +158,12 @@ class ErasureCodeDummyImpl : public ErasureCodeInterface {
       // for each missing shard.
       for (auto a : available) {
         minimum_set.insert(a);
-        if (minimum_set.size() == data_chunk_count) {
+        if (std::cmp_equal(minimum_set.size(), data_chunk_count)) {
           break;
         }
       }
 
-      if (minimum_set.size() != data_chunk_count) {
+      if (std::cmp_not_equal(minimum_set.size(), data_chunk_count)) {
         minimum_set.clear();
         return -EIO; // Cannot recover.
       }
@@ -174,7 +174,6 @@ class ErasureCodeDummyImpl : public ErasureCodeInterface {
     }
     return 0;
   }
-
   [[deprecated]]
   int minimum_to_decode(const std::set<int> &want_to_read,
     const std::set<int> &available,

src/test/osd/test_scrubber_be.cc

@@ -118,7 +118,7 @@ class TestPg : public PgScrubBeListener {
     }
 
     for (shard_id_t i; i < get_ec_sinfo().get_k(); ++i) {
-      for (int j = 0; j < get_ec_sinfo().get_chunk_size(); j++) {
+      for (int j = 0; std::cmp_less(j, get_ec_sinfo().get_chunk_size()); j++) {
         encode_map.at(i).c_str()[j] =
             chunks[j + (get_ec_sinfo().get_chunk_size() * i.id)];
         for (shard_id_t k{static_cast<int8_t>(get_ec_sinfo().get_k_plus_m())};