commit 05b243aff343fd6761bbadb2fcb4d2d98b0848c9
author Sheri Zhang <sheri.zhang@arm.com> Mon Mar 16 14:07:51 2020 +0000
committer Sheri Zhang <sheri.zhang@arm.com> Mon Mar 16 16:35:33 2020 +0000
tree f0a99e11ae4990aa68881e558b0fd86ca8a1978e
parent a602f03f4c66e5ee2480f1a3fc66847968fc1076

COMPMID-3271: Add support for QASYMM8_SIGNED in CPPTopKVKernel/CPPTopKV

Signed-off-by: Sheri Zhang <sheri.zhang@arm.com>
Change-Id: Ic34616fc3480ca85cc582e4e3db031d631ed5861
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/2887
Reviewed-by: Michele Di Giorgio <michele.digiorgio@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>

src/core/CPP/kernels/CPPTopKVKernel.cpp

diff --git a/src/core/CPP/kernels/CPPTopKVKernel.cpp b/src/core/CPP/kernels/CPPTopKVKernel.cpp
index 533543a..7f284d4 100644
--- a/src/core/CPP/kernels/CPPTopKVKernel.cpp
+++ b/src/core/CPP/kernels/CPPTopKVKernel.cpp
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2019 ARM Limited.
+ * Copyright (c) 2019-2020 ARM Limited.
  *
  * SPDX-License-Identifier: MIT
  *
@@ -54,7 +54,7 @@
 Status validate_arguments(const ITensorInfo *predictions, const ITensorInfo *targets, ITensorInfo *output, const unsigned int k)
 {
     ARM_COMPUTE_UNUSED(k);
-    ARM_COMPUTE_RETURN_ERROR_ON_DATA_TYPE_CHANNEL_NOT_IN(predictions, 1, DataType::QASYMM8, DataType::S32, DataType::F16, DataType::F32);
+    ARM_COMPUTE_RETURN_ERROR_ON_DATA_TYPE_CHANNEL_NOT_IN(predictions, 1, DataType::QASYMM8, DataType::QASYMM8_SIGNED, DataType::S32, DataType::F16, DataType::F32);
     ARM_COMPUTE_RETURN_ERROR_ON_DATA_TYPE_CHANNEL_NOT_IN(targets, 1, DataType::U32);
 
     ARM_COMPUTE_RETURN_ERROR_ON(predictions->num_dimensions() > 2);
@@ -145,6 +145,9 @@
         case DataType::QASYMM8:
             run_topkv<uint8_t>();
             break;
+        case DataType::QASYMM8_SIGNED:
+            run_topkv<int8_t>();
+            break;
         default:
             ARM_COMPUTE_ERROR("Not supported");
     }