aboutsummaryrefslogtreecommitdiff
path: root/src/core/CL/CLUtils.cpp
diff options
context:
space:
mode:
authorSiCongLi <sicong.li@arm.com>2021-11-03 19:01:22 +0000
committerSiCong Li <sicong.li@arm.com>2021-11-04 14:03:19 +0000
commitd928735fee6baefdb74325c05d8152dd13044f32 (patch)
tree6fb702e36da2863639149995e2df1cfe70905fc7 /src/core/CL/CLUtils.cpp
parent6049edadf0c89a026b3fcd1927ee7531d3c40278 (diff)
downloadComputeLibrary-d928735fee6baefdb74325c05d8152dd13044f32.tar.gz
Add validate tests for CLConvolutionLayer and CLGEMMConvolutionLayer with post ops
* Add validate tests * Restrict post ops support in ClGemmConv2d to only those that do not need im2col or col2im. In practice this means we only support post ops in conv1x1 with stride = 1, dilation = 1 and data layout = NHWC Resolves COMPMID-4435 Change-Id: I1fdf0c5d565a4624857250075ac76db35c2f383b Signed-off-by: SiCongLi <sicong.li@arm.com> Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/6573 Reviewed-by: Gunes Bayir <gunes.bayir@arm.com> Comments-Addressed: Arm Jenkins <bsgcomp@arm.com> Tested-by: Arm Jenkins <bsgcomp@arm.com>
Diffstat (limited to 'src/core/CL/CLUtils.cpp')
-rw-r--r--src/core/CL/CLUtils.cpp6
1 files changed, 3 insertions, 3 deletions
diff --git a/src/core/CL/CLUtils.cpp b/src/core/CL/CLUtils.cpp
index 88b31c8349..8dab8aa876 100644
--- a/src/core/CL/CLUtils.cpp
+++ b/src/core/CL/CLUtils.cpp
@@ -96,9 +96,9 @@ bool PostOpCLKernelUtils::are_post_op_shapes_compliant(const ITensorInfo *dst, c
return false;
}
// NOTE: Kernel limitation: currently only the following broadcasting types are supported:
- // 1. Post op arg is scalar, broadcast in both X and Y
- // 2. Post op arg is of shape: Y=1, X=N, broadcast only in Y
- // This means this case: Post op arg is of shape: Y=M, X=1, broadcast only in X, is NOT supported
+ // 1. Post op arg is scalar, broadcast in both first and second dims
+ // 2. Post op arg is of shape: second dim=1, first dim=N, broadcast only in second dim
+ // This means this case: Post op arg is of shape: second dim=M, first dim=1, broadcast only in first dim, is NOT supported
if(dst->dimension(0) > 1 && dst->dimension(1) > 1 && (*tensor)->dimension(0) == 1 && (*tensor)->dimension(1) > 1)
{
return false;