Avoided unnecessary memory alloc.

dendibakh · Dec 17, 2024 · c86b55b · c86b55b
1 parent 699476d
commit c86b55b
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/labs/memory_bound/loop_interchange_2/solution.cpp b/labs/memory_bound/loop_interchange_2/solution.cpp
@@ -48,16 +48,15 @@ static void filterVertically(uint8_t *output, const uint8_t *input,
     }
   }
 
-  std::unique_ptr<int[]> dot = std::make_unique<int[]>(width * (height - 2 * radius));
   // Accum
   for (int r = radius; r < height - radius; r++) {
       for (int c = 0; c < width; c++) { 
+        int dot = 0;
         for (int i = 0; i < radius + 1 + radius; i++) {
-              dot[(r - radius) * width + c] += input[(r - radius + i) * width + c] * kernel[i];
+              dot += input[(r - radius + i) * width + c] * kernel[i];
         }
         // Output
-        int dotIndex = (r - radius) * width + c;
-        int value = (dot[dotIndex] + rounding) >> shift;
+        int value = (dot + rounding) >> shift;
         output[r * width + c] = static_cast<uint8_t>(value);
       }
   }