diff options
author | Adnan AlSinan <adnan.alsinan@arm.com> | 2022-08-10 18:14:33 +0100 |
---|---|---|
committer | Adnan AlSinan <adnan.alsinan@arm.com> | 2022-08-11 13:41:48 +0000 |
commit | cec1af5cefd240ee133c2877aed8f09c24b99dd6 (patch) | |
tree | a24a39b7ae7c783a5542e19915aa59a592ea761a /src/gpu/cl/operators | |
parent | cc03419bd2f7dd12d1c130034f64962dfcb9348e (diff) | |
download | ComputeLibrary-cec1af5cefd240ee133c2877aed8f09c24b99dd6.tar.gz |
Fix performance regression in Conv2D on OpenCL
- Affect CL backend.
- For FP32 datatype, affect different platforms.
Resolves COMPMID-5479
Signed-off-by: Adnan AlSinan <adnan.alsinan@arm.com>
Change-Id: I705d718bc9b7def218034958f7ef86f2c2abe06d
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/8064
Benchmark: Arm Jenkins <bsgcomp@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Gian Marco Iodice <gianmarco.iodice@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Diffstat (limited to 'src/gpu/cl/operators')
-rw-r--r-- | src/gpu/cl/operators/ClConv2d.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/gpu/cl/operators/ClConv2d.cpp b/src/gpu/cl/operators/ClConv2d.cpp index db193ceb01..8119fc8e3d 100644 --- a/src/gpu/cl/operators/ClConv2d.cpp +++ b/src/gpu/cl/operators/ClConv2d.cpp @@ -281,7 +281,7 @@ ConvolutionMethod ClConv2d::get_convolution_method(const ITensorInfo *src, const } // Direct convolution case - if(is_direct_valid && workload_gte_8192) + if(is_direct_valid) { if((gpu_target == arm_compute::GPUTarget::G71 || gpu_target == arm_compute::GPUTarget::G72 || get_arch_from_target(gpu_target) == arm_compute::GPUTarget::MIDGARD)) { @@ -292,14 +292,14 @@ ConvolutionMethod ClConv2d::get_convolution_method(const ITensorInfo *src, const } else if(gpu_target == arm_compute::GPUTarget::G76) { - if((is_large_kernel_sz && is_ifm_ge_16) || (is_ofm_lte_8 && is_ifm_ge_16)) + if((is_large_kernel_sz && workload_gte_8192 && is_ifm_ge_16) || (is_ofm_lte_8 && is_ifm_ge_16)) { return ConvolutionMethod::DIRECT; } } else { - if(is_large_kernel_sz || is_ofm_lte_8 || is_m_one) + if( ((is_large_kernel_sz || is_m_one) && workload_gte_8192) || is_ofm_lte_8 ) { return ConvolutionMethod::DIRECT; } |