diff options
author | SiCongLi <sicong.li@arm.com> | 2021-12-22 11:22:40 +0000 |
---|---|---|
committer | SiCong Li <sicong.li@arm.com> | 2021-12-25 10:54:51 +0000 |
commit | c4270cf958e85e0c41590030e1f9e228493a5ba0 (patch) | |
tree | fe26e2724df1d9da12c8462a576688b93838bc79 /src/core/Utils.cpp | |
parent | cb86956e1972be4b2ddbaacaa23a0d21185f8ccb (diff) | |
download | ComputeLibrary-c4270cf958e85e0c41590030e1f9e228493a5ba0.tar.gz |
Add tests for FP Cpu Pooling where pool region is completely outside the input
* Add floating point validation tests for this configuration
* Fix reference implementation to return -inf for this configuration
* Prohibit this config in Cl, as well as non-float cases in Cpu
* Direct this config to non-asm path
Resolves COMPMID-4998
Change-Id: If88025c51b14ea337aea2441c548f858e95e5819
Signed-off-by: SiCongLi <sicong.li@arm.com>
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/6857
Reviewed-by: Gunes Bayir <gunes.bayir@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Diffstat (limited to 'src/core/Utils.cpp')
-rw-r--r-- | src/core/Utils.cpp | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/src/core/Utils.cpp b/src/core/Utils.cpp index 930e4c7975..ef16cb5e75 100644 --- a/src/core/Utils.cpp +++ b/src/core/Utils.cpp @@ -232,6 +232,18 @@ const std::string &string_from_pooling_type(PoolingType type) return pool_type_map[type]; } +bool is_pool_region_entirely_outside_input(const PoolingLayerInfo &info) +{ + if(info.is_global_pooling || info.exclude_padding || info.pool_size.x() == 0 || info.pool_size.y() == 0) + { + return false; + } + const auto ps = info.pad_stride_info; + const auto pool_le_padding_x = info.pool_size.x() <= std::max({ ps.pad_left(), ps.pad_right() }); + const auto pool_le_padding_y = info.pool_size.y() <= std::max({ ps.pad_top(), ps.pad_bottom() }); + return pool_le_padding_x || pool_le_padding_y; +} + const std::string &string_from_gemmlowp_output_stage(GEMMLowpOutputStageType output_stage) { static std::map<GEMMLowpOutputStageType, const std::string> output_stage_map = |