Fix two warnings on GCC 14.2.1. (#8430)

halide · Oct 11, 2024 · 9a55d9c · 9a55d9c
1 parent c4199ad
commit 9a55d9c
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 10 deletions.
diff --git a/src/Derivative.cpp b/src/Derivative.cpp
@@ -1816,8 +1816,8 @@ void ReverseAccumulationVisitor::propagate_halide_function_call(
         }
         // If previous update has a different set of reduction variables,
         // don't merge
-        const vector<ReductionVariable> &rvars =
-            func_to_update.function().update(update_id).schedule().rvars();
+        Function func = func_to_update.function();
+        const vector<ReductionVariable> &rvars = func.update(update_id).schedule().rvars();
         if (!merged_r.defined()) {
             return rvars.empty();
         }

diff --git a/src/runtime/HalideBuffer.h b/src/runtime/HalideBuffer.h
@@ -1431,23 +1431,23 @@ class Buffer {
         // into a static dispatch to the right-sized copy.)
         if (T_is_void ? (type().bytes() == 1) : (sizeof(not_void_T) == 1)) {
             using MemType = uint8_t;
-            auto &typed_dst = (Buffer<MemType, Dims, InClassDimStorage> &)dst;
-            auto &typed_src = (Buffer<const MemType, D2, S2> &)src;
+            auto &typed_dst = reinterpret_cast<Buffer<MemType, Dims, InClassDimStorage> &>(dst);
+            auto &typed_src = reinterpret_cast<Buffer<const MemType, D2, S2> &>(src);
             typed_dst.for_each_value([&](MemType &dst, MemType src) { dst = src; }, typed_src);
         } else if (T_is_void ? (type().bytes() == 2) : (sizeof(not_void_T) == 2)) {
             using MemType = uint16_t;
-            auto &typed_dst = (Buffer<MemType, Dims, InClassDimStorage> &)dst;
-            auto &typed_src = (Buffer<const MemType, D2, S2> &)src;
+            auto &typed_dst = reinterpret_cast<Buffer<MemType, Dims, InClassDimStorage> &>(dst);
+            auto &typed_src = reinterpret_cast<Buffer<const MemType, D2, S2> &>(src);
             typed_dst.for_each_value([&](MemType &dst, MemType src) { dst = src; }, typed_src);
         } else if (T_is_void ? (type().bytes() == 4) : (sizeof(not_void_T) == 4)) {
             using MemType = uint32_t;
-            auto &typed_dst = (Buffer<MemType, Dims, InClassDimStorage> &)dst;
-            auto &typed_src = (Buffer<const MemType, D2, S2> &)src;
+            auto &typed_dst = reinterpret_cast<Buffer<MemType, Dims, InClassDimStorage> &>(dst);
+            auto &typed_src = reinterpret_cast<Buffer<const MemType, D2, S2> &>(src);
             typed_dst.for_each_value([&](MemType &dst, MemType src) { dst = src; }, typed_src);
         } else if (T_is_void ? (type().bytes() == 8) : (sizeof(not_void_T) == 8)) {
             using MemType = uint64_t;
-            auto &typed_dst = (Buffer<MemType, Dims, InClassDimStorage> &)dst;
-            auto &typed_src = (Buffer<const MemType, D2, S2> &)src;
+            auto &typed_dst = reinterpret_cast<Buffer<MemType, Dims, InClassDimStorage> &>(dst);
+            auto &typed_src = reinterpret_cast<Buffer<const MemType, D2, S2> &>(src);
             typed_dst.for_each_value([&](MemType &dst, MemType src) { dst = src; }, typed_src);
         } else {
             assert(false && "type().bytes() must be 1, 2, 4, or 8");