Things still work, undoing uneeded source code changes

Author: BujSet

kernels/portable/cpu/op_repeat_interleave.cpp

@@ -50,7 +50,7 @@ bool check_repeat_interleave_args(
     for (const auto i : c10::irange(repeats.numel())) {
       ET_CHECK_OR_RETURN_FALSE(
           repeats_data[i] >= 0,
-          "repeats cannot be negative; repeats_data[%" PRId64 "] = %ld",
+          "repeats cannot be negative; repeats_data[%" PRId64 "] = %d",
           static_cast<int64_t>(i),
           repeats_data[i]);
     }

kernels/portable/cpu/util/math_util.h

@@ -183,7 +183,7 @@ template <
     typename std::enable_if<std::is_floating_point<CTYPE>::value, int>::type =
         0>
 CTYPE remainder_override(CTYPE a, CTYPE b) {
-  CTYPE rem = std::fmod(a, b);
+  float rem = std::fmod(a, b);
   if (((a < 0) ^ (b < 0)) && rem != 0) {
     rem += b;
   }

runtime/executor/pte_data_map.cpp

@@ -32,7 +32,7 @@ PteDataMap::get_data(executorch::aten::string_view key) const {
     ET_CHECK_OR_RETURN_ERROR(
         named_data_->Get(i) != nullptr && named_data_->Get(i)->key() != nullptr,
         InvalidArgument,
-        "Searching for key %.*s: NamedData at index %lu is null",
+        "Searching for key %.*s: NamedData at index %d is null",
         static_cast<int>(key.size()),
         key.data(),
         i);
@@ -47,7 +47,7 @@ PteDataMap::get_data(executorch::aten::string_view key) const {
       ET_CHECK_OR_RETURN_ERROR(
           segment_index < segments_->size(),
           InvalidArgument,
-          "Segment index %zu for key %.*s is out of range for segments size %lu",
+          "Segment index %zu for key %.*s is out of range for segments size %u",
           segment_index,
           static_cast<int>(key.size()),
           key.data(),
@@ -74,15 +74,15 @@ ET_NODISCARD executorch::runtime::Result<const char*> PteDataMap::get_key(
   ET_CHECK_OR_RETURN_ERROR(
       index < named_data_->size(),
       InvalidArgument,
-      "Index out of range: named_data size is %lu, received index %lu",
+      "Index out of range: named_data size is %u, received index %u",
       named_data_->size(),
       index);
 
   ET_CHECK_OR_RETURN_ERROR(
       named_data_->Get(index) != nullptr &&
           named_data_->Get(index)->key() != nullptr,
       InvalidArgument,
-      "NamedData at index %lu is null",
+      "NamedData at index %u is null",
       index);
   return named_data_->Get(index)->key()->c_str();
 }

runtime/executor/tensor_parser_exec_aten.cpp

@@ -133,7 +133,7 @@ ET_NODISCARD Error validateTensorLayout(
     ET_CHECK_OR_RETURN_ERROR(
         s_tensor->sizes()->Get(i) == expected_layout.sizes()[i],
         InvalidExternalData,
-        "Sizes mismatch. Expected %ld, got %ld for size at index %d.",
+        "Sizes mismatch. Expected %d, got %d for size at index %d.",
         s_tensor->sizes()->Get(i),
         expected_layout.sizes()[i],
         i);

runtime/kernel/operator_registry.h

@@ -34,7 +34,7 @@
   for (const auto& meta : meta_list) {                               \
     ET_LOG(Info, "dtype: %d | dim order: [", int(meta.dtype_));      \
     for (size_t i = 0; i < meta.dim_order_.size(); i++) {            \
-      ET_LOG(Info, "%ld,", static_cast<int32_t>(meta.dim_order_[i])); \
+      ET_LOG(Info, "%d,", static_cast<int32_t>(meta.dim_order_[i])); \
     }                                                                \
     ET_LOG(Info, "]");                                               \
   }