diff options
author | Georgios Pinitas <georgios.pinitas@arm.com> | 2020-06-02 23:00:41 +0100 |
---|---|---|
committer | Georgios Pinitas <georgios.pinitas@arm.com> | 2020-06-08 16:42:03 +0000 |
commit | 25ef7217ec4e13682bf37c87c0c6075a799ba1c0 (patch) | |
tree | 8ed3e4d7b811a36322560298ab6c8a0484d10ab8 /arm_compute/core/NEON/kernels | |
parent | ebfb2f8701c131294b3c5c1e36547fa3658d09dd (diff) | |
download | ComputeLibrary-25ef7217ec4e13682bf37c87c0c6075a799ba1c0.tar.gz |
COMPMID-3180: Remove padding from NEThreshold
- Removes padding from NEThresholdKernel
- Alters configuration interface to use a descriptor
Change-Id: I394d5e1375454813856d9d206e61dc9a87c2cadc
Signed-off-by: Georgios Pinitas <georgios.pinitas@arm.com>
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/3300
Reviewed-by: Michele Di Giorgio <michele.digiorgio@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Diffstat (limited to 'arm_compute/core/NEON/kernels')
-rw-r--r-- | arm_compute/core/NEON/kernels/NEThresholdKernel.h | 39 |
1 files changed, 19 insertions, 20 deletions
diff --git a/arm_compute/core/NEON/kernels/NEThresholdKernel.h b/arm_compute/core/NEON/kernels/NEThresholdKernel.h index a6d1e9071c..fc97c3a75f 100644 --- a/arm_compute/core/NEON/kernels/NEThresholdKernel.h +++ b/arm_compute/core/NEON/kernels/NEThresholdKernel.h @@ -1,5 +1,5 @@ /* - * Copyright (c) 2016-2019 ARM Limited. + * Copyright (c) 2016-2020 ARM Limited. * * SPDX-License-Identifier: MIT * @@ -24,18 +24,15 @@ #ifndef ARM_COMPUTE_NETHRESHOLDKERNEL_H #define ARM_COMPUTE_NETHRESHOLDKERNEL_H +#include "arm_compute/core/KernelDescriptors.h" #include "arm_compute/core/NEON/INEKernel.h" #include "arm_compute/core/Types.h" -#include <cstdint> - namespace arm_compute { class ITensor; -/** Interface for the thresholding kernel - * - */ +/** Interface for the thresholding kernel */ class NEThresholdKernel : public INEKernel { public: @@ -53,15 +50,20 @@ public: NEThresholdKernel &operator=(const NEThresholdKernel &) = delete; /** Initialise the kernel's input, output and threshold parameters. * - * @param[in] input An input tensor. Data type supported: U8 - * @param[out] output The output tensor. Data type supported: U8. - * @param[in] threshold Threshold. When the threhold type is RANGE, this is used as the lower threshold. - * @param[in] false_value value to set when the condition is not respected. - * @param[in] true_value value to set when the condition is respected. - * @param[in] type Thresholding type. Either RANGE or BINARY. - * @param[in] upper Upper threshold. Only used when the thresholding type is RANGE. + * @param[in] input An input tensor. Data type supported: U8 + * @param[out] output The output tensor. Data type supported: U8. + * @param[in] info Threshold kernel descriptor + */ + void configure(const ITensor *input, ITensor *output, const ThresholdKernelInfo &info); + /** Static function to check if given info will lead to a valid configuration of @ref NEThresholdKernel + * + * @param[in] input Input tensor info. Data type supported: U8 + * @param[in] output Output tensor info. Data type supported: U8 + * @param[in] info Threshold kernel descriptor + * + * @return A status containing an error code in case of failure */ - void configure(const ITensor *input, ITensor *output, uint8_t threshold, uint8_t false_value, uint8_t true_value, ThresholdType type, uint8_t upper); + static Status validate(const ITensorInfo *input, const ITensorInfo *output, const ThresholdKernelInfo &info); // Inherited methods overridden: void run(const Window &window, const ThreadInfo &info) override; @@ -74,12 +76,9 @@ private: void (NEThresholdKernel::*_func)(const Window &window); - const ITensor *_input; /**< Input */ - ITensor *_output; /**< Output */ - uint8_t _threshold; - uint8_t _false_value; - uint8_t _true_value; - uint8_t _upper; + const ITensor *_input; /**< Input */ + ITensor *_output; /**< Output */ + ThresholdKernelInfo _info; /**< Threshold descriptor */ }; } // namespace arm_compute #endif /*ARM_COMPUTE_NETHRESHOLDKERNEL_H */ |