diff options
author | Georgios Pinitas <georgios.pinitas@arm.com> | 2017-11-29 14:27:24 +0000 |
---|---|---|
committer | Anthony Barbier <anthony.barbier@arm.com> | 2018-11-02 16:41:36 +0000 |
commit | 6fdfaa856b1eb69d6afbef5727b99756912fc6fa (patch) | |
tree | 1f1b916fc94a140ec16090a638f9d0531032d9bb /src/core/CL/cl_kernels/direct_convolution_1x1_3x3_5x5_quantized.cl | |
parent | 47b5603a0db2797dda66a2b5dbbc451a740a5ecd (diff) | |
download | ComputeLibrary-6fdfaa856b1eb69d6afbef5727b99756912fc6fa.tar.gz |
COMPMID-713: Address failures in OCLGrind for CLDirectConvolution
-Changes way of clamping in the kernel side.
-Fills padding with quantized values
Change-Id: I94d17c341fd637fbb24390722162b551b62d16cb
Reviewed-on: https://eu-gerrit-1.euhpc.arm.com/111114
Reviewed-by: Anthony Barbier <anthony.barbier@arm.com>
Tested-by: BSG Visual Compute Jenkins server to access repositories on http://mpd-gerrit.cambridge.arm.com <bsgcomp@arm.com>
Diffstat (limited to 'src/core/CL/cl_kernels/direct_convolution_1x1_3x3_5x5_quantized.cl')
-rw-r--r-- | src/core/CL/cl_kernels/direct_convolution_1x1_3x3_5x5_quantized.cl | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/core/CL/cl_kernels/direct_convolution_1x1_3x3_5x5_quantized.cl b/src/core/CL/cl_kernels/direct_convolution_1x1_3x3_5x5_quantized.cl index cbe826639d..d0cf032ac1 100644 --- a/src/core/CL/cl_kernels/direct_convolution_1x1_3x3_5x5_quantized.cl +++ b/src/core/CL/cl_kernels/direct_convolution_1x1_3x3_5x5_quantized.cl @@ -243,9 +243,7 @@ __kernel void direct_convolution_1x1_3x3_5x5_quantized( pixels0 = ASYMM_MULT_BY_QUANT_MULTIPLIER_LESS_THAN_ONE(pixels0, output_multiplier, output_shift, 8); pixels0 = pixels0 + output_offset; - pixels0 = max(pixels0, 0); - pixels0 = min(pixels0, 255); - vstore8(convert_uchar8(pixels0), 0, (__global uchar *)dst.ptr); + vstore8(convert_uchar8_sat(pixels0), 0, (__global uchar *)dst.ptr); } #endif // defined(DATA_TYPE) && defined(STRIDE_X) && defined(WEIGHTS_DEPTH) |