diff options
author | SiCongLi <sicong.li@arm.com> | 2021-10-29 15:05:49 +0100 |
---|---|---|
committer | SiCong Li <sicong.li@arm.com> | 2021-11-01 14:29:51 +0000 |
commit | eb8bd81a625f0f87080dbde55b434362ad57324a (patch) | |
tree | fda1de0843be17266388d0d137908f392a7f694e /src/core/experimental | |
parent | 1af5416917268692fcd4b34b1d7ffebd3a2aea8a (diff) | |
download | ComputeLibrary-eb8bd81a625f0f87080dbde55b434362ad57324a.tar.gz |
Fix dst "widening" validation
* Auto-initialize the dst tensor before checking for PostOp shape
compliance so that we catch the invalid case of "widening" dst tensor
shape
* Rework post op validate test cases to be more readable
Partially resolves: COMPMID-4435
Change-Id: I79943994182942f962e4d59a7fa0d6f017ae9ac7
Signed-off-by: SiCongLi <sicong.li@arm.com>
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/6548
Reviewed-by: Gian Marco Iodice <gianmarco.iodice@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Diffstat (limited to 'src/core/experimental')
-rw-r--r-- | src/core/experimental/PostOp.h | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/core/experimental/PostOp.h b/src/core/experimental/PostOp.h index 64414d2050..7d62bd95e1 100644 --- a/src/core/experimental/PostOp.h +++ b/src/core/experimental/PostOp.h @@ -79,9 +79,9 @@ template <typename TensorRelatedT> struct PostOpEltwiseAdd : public IPostOp<TensorRelatedT> { public: - PostOpEltwiseAdd(TensorRelatedT addend, int prev_op_arg_pos, ConvertPolicy policy) + PostOpEltwiseAdd(TensorRelatedT addend, int prev_dst_pos, ConvertPolicy policy) : _addend{ addend }, - _prev_op_arg_pos{ prev_op_arg_pos }, + _prev_dst_pos{ prev_dst_pos }, _policy{ policy } { } @@ -93,7 +93,7 @@ public: PostOpEltwiseAdd &operator=(PostOpEltwiseAdd &&) = default; int prev_dst_pos() const override { - return _prev_op_arg_pos; + return _prev_dst_pos; } PostOpType type() const override { @@ -112,7 +112,7 @@ public: return std::make_unique<PostOpEltwiseAdd<TensorRelatedT>>(*this); } TensorRelatedT _addend; - int _prev_op_arg_pos; + int _prev_dst_pos; ConvertPolicy _policy; }; @@ -135,7 +135,7 @@ PostOpList<ToTensorT> transform_post_op_list_arguments(const PostOpList<FromTens case PostOpType::Eltwise_Add: { const auto _post_op = utils::cast::polymorphic_downcast<const PostOpEltwiseAdd<FromTensorT> *>(post_op.get()); - transformed_post_ops.template push_back_op<PostOpEltwiseAdd<ToTensorT>>(transform_arg(_post_op->_addend), _post_op->_prev_op_arg_pos, _post_op->_policy); + transformed_post_ops.template push_back_op<PostOpEltwiseAdd<ToTensorT>>(transform_arg(_post_op->_addend), _post_op->_prev_dst_pos, _post_op->_policy); break; } default: |