From b0cd5d8cf6945bed4df62603799d73b5413654a1 Mon Sep 17 00:00:00 2001 From: Georgios Pinitas Date: Mon, 22 Feb 2021 18:17:43 +0000 Subject: Use polymorphic_downcast instead of dynamic_cast Signed-off-by: Georgios Pinitas Change-Id: Ibd8f629c3f0a2943705ba0f7d65a264948dac4b0 Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/5147 Tested-by: Arm Jenkins Reviewed-by: Michele Di Giorgio Comments-Addressed: Arm Jenkins --- src/runtime/CL/mlgo/HeuristicTree.cpp | 8 +++++--- src/runtime/OperatorTensor.cpp | 6 ++++-- 2 files changed, 9 insertions(+), 5 deletions(-) (limited to 'src/runtime') diff --git a/src/runtime/CL/mlgo/HeuristicTree.cpp b/src/runtime/CL/mlgo/HeuristicTree.cpp index 65219998cb..1c75cdc427 100644 --- a/src/runtime/CL/mlgo/HeuristicTree.cpp +++ b/src/runtime/CL/mlgo/HeuristicTree.cpp @@ -24,6 +24,8 @@ #include "src/runtime/CL/mlgo/HeuristicTree.h" #include "arm_compute/core/Log.h" +#include "support/Cast.h" + #include #include #include @@ -114,7 +116,7 @@ std::pair HeuristicTree::query(GEMMShape shape) const return std::make_pair(false, T{}); } ARM_COMPUTE_ERROR_ON_MSG(cur_node->type() != NodeType::Branch, "Unexpected NodeType"); - auto br_node = dynamic_cast(cur_node); + auto br_node = utils::cast::polymorphic_downcast(cur_node); if(evaluate(shape, br_node->condition)) { cur_node = _tree.at(br_node->true_node).get(); @@ -126,7 +128,7 @@ std::pair HeuristicTree::query(GEMMShape shape) const ++depth; } ARM_COMPUTE_ERROR_ON_MSG(cur_node->type() != NodeType::Leaf, "Unexpected NodeType"); - auto l_node = dynamic_cast *>(cur_node); + auto l_node = utils::cast::polymorphic_downcast *>(cur_node); return std::make_pair(true, l_node->value); } @@ -202,7 +204,7 @@ bool HeuristicTree::check_if_structurally_correct() const auto cur_node = _tree.at(id).get(); if(cur_node->type() == NodeType::Branch) { - auto br_node = dynamic_cast(cur_node); + auto br_node = utils::cast::polymorphic_downcast(cur_node); to_visit.push_back(br_node->true_node); to_visit.push_back(br_node->false_node); } diff --git a/src/runtime/OperatorTensor.cpp b/src/runtime/OperatorTensor.cpp index 5a35154479..a8ad53da90 100644 --- a/src/runtime/OperatorTensor.cpp +++ b/src/runtime/OperatorTensor.cpp @@ -1,5 +1,5 @@ /* - * Copyright (c) 2020 Arm Limited. + * Copyright (c) 2020-2021 Arm Limited. * * SPDX-License-Identifier: MIT * @@ -24,6 +24,8 @@ #include "arm_compute/runtime/OperatorTensor.h" #include "arm_compute/runtime/MemoryRegion.h" +#include "support/Cast.h" + namespace arm_compute { namespace experimental @@ -48,7 +50,7 @@ uint8_t *OperatorTensor::buffer() const switch(_mem_type) { case MemoryType::CPU: - return (uint8_t *)dynamic_cast(_memory->region())->buffer(); + return (uint8_t *)utils::cast::polymorphic_downcast(_memory->region())->buffer(); default: ARM_COMPUTE_ERROR("Memory type not supported."); } -- cgit v1.2.1