diff options
author | Giorgio Arena <giorgio.arena@arm.com> | 2021-01-21 14:53:56 +0000 |
---|---|---|
committer | Giorgio Arena <giorgio.arena@arm.com> | 2021-01-22 09:12:01 +0000 |
commit | e36208c8b86413e4fdd4ca31904e9d613ce11570 (patch) | |
tree | af6b3f363b7f5b00308c1706275c3b91381f08be /src/core/CL/kernels/CLArgMinMaxLayerKernel.cpp | |
parent | 0094c023038cbb353f60e96d1301b4bb25c9e382 (diff) | |
download | ComputeLibrary-e36208c8b86413e4fdd4ca31904e9d613ce11570.tar.gz |
CTS failures in Android R and Q in GpuAcc in ArgMinMax
- Fix ambiguosity with select in OpenCL
- Define a new macro for signed integer data type of the same input data type's size. This is needed because some ops (e.g. logical operators) in OpenCL work in this way
Resolves: COMPMID-4116, COMPMID-4110
Signed-off-by: Giorgio Arena <giorgio.arena@arm.com>
Change-Id: I560eda63fce24abd03d061f78f2f2ca951053fd0
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/4898
Reviewed-by: Michele Di Giorgio <michele.digiorgio@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Diffstat (limited to 'src/core/CL/kernels/CLArgMinMaxLayerKernel.cpp')
-rw-r--r-- | src/core/CL/kernels/CLArgMinMaxLayerKernel.cpp | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/core/CL/kernels/CLArgMinMaxLayerKernel.cpp b/src/core/CL/kernels/CLArgMinMaxLayerKernel.cpp index 8215d3ce07..909972482f 100644 --- a/src/core/CL/kernels/CLArgMinMaxLayerKernel.cpp +++ b/src/core/CL/kernels/CLArgMinMaxLayerKernel.cpp @@ -105,7 +105,6 @@ void CLArgMinMaxLayerKernel::configure(const CLCompileContext &compile_context, build_opts.add_option_if(is_data_type_float(input->info()->data_type()), "-DFLOAT_DATA_TYPE"); build_opts.add_option_if_else(op == ReductionOperation::ARG_IDX_MAX, "-DARG_MAX", "-DARG_MIN"); build_opts.add_option("-DDATA_TYPE_OUTPUT=" + get_cl_type_from_data_type(output->info()->data_type())); - build_opts.add_option("-DDATA_TYPE_SELECT=" + get_cl_signed_type_from_element_size(input->info()->element_size())); // Create kernel cl::NDRange lws_hint = CLKernelLibrary::get().default_ndrange(); |