diff options
author | Georgios Pinitas <georgios.pinitas@arm.com> | 2018-07-10 17:03:11 +0100 |
---|---|---|
committer | Anthony Barbier <anthony.barbier@arm.com> | 2018-11-02 16:54:10 +0000 |
commit | ed32f43174ce45cafe9d93e1a0b92cbebbcb7493 (patch) | |
tree | e778781b63b39a2e591a526dd8522c86dfafd841 /src/core/CL/cl_kernels | |
parent | cdb43bda919a7df4a3829fa9d51cbffa24fa3e14 (diff) | |
download | ComputeLibrary-ed32f43174ce45cafe9d93e1a0b92cbebbcb7493.tar.gz |
COMPMID-1383: OCLGrind failure in CLDepthwiseConvolution 3x3 stride 1 NHWC
Seems OCLGrind to operate wrongly on some intrinsics when there is a
mixture of vectors and scalars passed to it.
Change-Id: I9e3782e739603ec59bacc3c77d91a70b1899fe3e
Reviewed-on: https://eu-gerrit-1.euhpc.arm.com/139474
Tested-by: Jenkins <bsgcomp@arm.com>
Reviewed-by: Anthony Barbier <anthony.barbier@arm.com>
Diffstat (limited to 'src/core/CL/cl_kernels')
-rw-r--r-- | src/core/CL/cl_kernels/depthwise_convolution.cl | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/src/core/CL/cl_kernels/depthwise_convolution.cl b/src/core/CL/cl_kernels/depthwise_convolution.cl index 9a8b57e4c4..77a76b6a9f 100644 --- a/src/core/CL/cl_kernels/depthwise_convolution.cl +++ b/src/core/CL/cl_kernels/depthwise_convolution.cl @@ -1083,7 +1083,7 @@ __kernel void depthwise_convolution_3x3_nhwc( z_coord = z * CONV_STRIDE_Y - (int)CONV_PAD_TOP; z_coord = min((uint)z_coord, (uint)SRC_DIM_2); offset = y_offset + (int4)(z_coord * src_stride_z); - offset = min(offset, max_offset); + offset = min(offset, (int4)max_offset); VEC_FLOAT values0 = VLOAD(VEC_SIZE)(0, (__global float *)(src_addr + offset.s0)); VEC_FLOAT values1 = VLOAD(VEC_SIZE)(0, (__global float *)(src_addr + offset.s1)); @@ -1102,7 +1102,7 @@ __kernel void depthwise_convolution_3x3_nhwc( // After z = 1 we can simply add src_stride_z to offset without updating z_coord // However offset can be out-of-bound so we need to check if it is greater than max_offset offset += (int4)src_stride_z; - offset = min(offset, max_offset); + offset = min(offset, (int4)max_offset); VEC_FLOAT values6 = VLOAD(VEC_SIZE)(0, (__global float *)(src_addr + offset.s0)); VEC_FLOAT values7 = VLOAD(VEC_SIZE)(0, (__global float *)(src_addr + offset.s1)); VEC_FLOAT values8 = VLOAD(VEC_SIZE)(0, (__global float *)(src_addr + offset.s2)); @@ -1190,7 +1190,7 @@ __kernel void depthwise_convolution_3x3_nhwc_stride1( int z_coord = 0; int4 offset = 0; - int4 y_offset = ((int4)(y * NUM_ROWS_PROCESSED) + (int4)(0, 1, 2, 3) - CONV_PAD_LEFT) * (int4)src_stride_y; + int4 y_offset = ((int4)(y * NUM_ROWS_PROCESSED) + (int4)(0, 1, 2, 3) - (int)CONV_PAD_LEFT) * (int4)src_stride_y; // We compute 2x2x2 [C,W,H] elements VEC_FLOAT acc0 = 0; @@ -1214,10 +1214,10 @@ __kernel void depthwise_convolution_3x3_nhwc_stride1( // Clamp z_coord as for z = 0, it can be negative // z_coord is casted to unsigned int in order to use just a min() operation // A "-1" 32 bit signed variable converted to unsigned gives 4294967295 - z_coord = z * NUM_PLANES_PROCESSED - (int)CONV_PAD_TOP; + z_coord = z * (int)NUM_PLANES_PROCESSED - (int)CONV_PAD_TOP; z_coord = min((uint)z_coord, (uint)SRC_DIM_2); offset = y_offset + (int4)(z_coord * src_stride_z); - offset = min(offset, max_offset); + offset = min(offset, (int4)max_offset); VEC_FLOAT values0 = VLOAD(VEC_SIZE)(0, (__global float *)(src_addr + offset.s0)); VEC_FLOAT values1 = VLOAD(VEC_SIZE)(0, (__global float *)(src_addr + offset.s1)); @@ -1227,7 +1227,7 @@ __kernel void depthwise_convolution_3x3_nhwc_stride1( // z == 1 // z_coord can be only negative for z = 0 so we do not need to clamp it // Moreover z_coord cannot be out-of-bound for z = 1 so we do not need to clamp the offset - z_coord = z * NUM_PLANES_PROCESSED - (int)CONV_PAD_TOP + 1; + z_coord = z * (int)NUM_PLANES_PROCESSED - (int)CONV_PAD_TOP + 1; offset = y_offset + (int4)(z_coord * src_stride_z); VEC_FLOAT values4 = VLOAD(VEC_SIZE)(0, (__global float *)(src_addr + offset.s0)); VEC_FLOAT values5 = VLOAD(VEC_SIZE)(0, (__global float *)(src_addr + offset.s1)); @@ -1238,7 +1238,7 @@ __kernel void depthwise_convolution_3x3_nhwc_stride1( // After z = 1 we can simply add src_stride_z to offset without updating z_coord // However offset can be out-of-bound so we need to check if it is greater than max_offset offset += (int4)src_stride_z; - offset = min(offset, max_offset); + offset = min(offset, (int4)max_offset); VEC_FLOAT values8 = VLOAD(VEC_SIZE)(0, (__global float *)(src_addr + offset.s0)); VEC_FLOAT values9 = VLOAD(VEC_SIZE)(0, (__global float *)(src_addr + offset.s1)); VEC_FLOAT values10 = VLOAD(VEC_SIZE)(0, (__global float *)(src_addr + offset.s2)); @@ -1247,8 +1247,8 @@ __kernel void depthwise_convolution_3x3_nhwc_stride1( // z == 3 // After z = 1 we can simply add src_stride_z to offset without updating z_coord // However offset can be out-of-bound so we need to check if it is greater than max_offset - offset += (int4)(src_stride_z); - offset = min(offset, max_offset); + offset += (int4)src_stride_z; + offset = min(offset, (int4)max_offset); VEC_FLOAT values12 = VLOAD(VEC_SIZE)(0, (__global float *)(src_addr + offset.s0)); VEC_FLOAT values13 = VLOAD(VEC_SIZE)(0, (__global float *)(src_addr + offset.s1)); VEC_FLOAT values14 = VLOAD(VEC_SIZE)(0, (__global float *)(src_addr + offset.s2)); |