aboutsummaryrefslogtreecommitdiff
path: root/arm_compute/core/WindowIterator.h
diff options
context:
space:
mode:
authorGeorgios Pinitas <georgios.pinitas@arm.com>2018-08-01 12:11:15 +0100
committerAnthony Barbier <anthony.barbier@arm.com>2018-11-02 16:54:54 +0000
commitfb0b2804057ebcd01a7fdb1b42f86726602cc646 (patch)
tree78c1bf35aa1247dbb54482268d270f5efec0dec6 /arm_compute/core/WindowIterator.h
parent8bc745dfd133f46e68edad511e2933a590602a24 (diff)
downloadComputeLibrary-fb0b2804057ebcd01a7fdb1b42f86726602cc646.tar.gz
COMPMID-1438: MobilenetV1 regression.
Alters the ending conditions for y dimension to use the actual end offset as a bound and not the actual y window as this could be the whole execution window and can lead to overlapped calculations across threads. Change-Id: Ic6642bbaa8e85d4a4034a44234d6cb3347a2f4ff Reviewed-on: https://eu-gerrit-1.euhpc.arm.com/142229 Tested-by: Jenkins <bsgcomp@arm.com> Reviewed-by: Anthony Barbier <anthony.barbier@arm.com>
Diffstat (limited to 'arm_compute/core/WindowIterator.h')
-rw-r--r--arm_compute/core/WindowIterator.h4
1 files changed, 2 insertions, 2 deletions
diff --git a/arm_compute/core/WindowIterator.h b/arm_compute/core/WindowIterator.h
index 13cb9cf2f4..8e58d0ce1c 100644
--- a/arm_compute/core/WindowIterator.h
+++ b/arm_compute/core/WindowIterator.h
@@ -218,7 +218,7 @@ private:
// Shouldn't be possible to reach that point and not have at least one entire row to process
ARM_COMPUTE_ERROR_ON(_w.y().end() == _position.y());
// No leftover: all the rows lefts to process are full width:
- iterate_over_dim1(_w.y().end());
+ iterate_over_dim1(end_y + _w.y().step());
}
else
{
@@ -229,7 +229,7 @@ private:
PRINTF("full rows\n");
//Switch to full row size:
on_new_row_size(_w[0].start(), _w.x().end());
- iterate_over_dim1(_w.y().end() - _w.y().step());
+ iterate_over_dim1(end_y);
}
PRINTF("Final leftover\n");