diff options
Diffstat (limited to 'src/c/AclContext.cpp')
-rw-r--r-- | src/c/AclContext.cpp | 40 |
1 files changed, 20 insertions, 20 deletions
diff --git a/src/c/AclContext.cpp b/src/c/AclContext.cpp index e88995bcff..c6c0820c92 100644 --- a/src/c/AclContext.cpp +++ b/src/c/AclContext.cpp @@ -1,5 +1,5 @@ /* - * Copyright (c) 2021 Arm Limited. + * Copyright (c) 2021,2023 Arm Limited. * * SPDX-License-Identifier: MIT * @@ -22,6 +22,7 @@ * SOFTWARE. */ #include "arm_compute/AclEntrypoints.h" +#include "arm_compute/core/Error.h" #include "src/common/IContext.h" #include "src/common/utils/Macros.h" @@ -40,23 +41,25 @@ namespace template <typename ContextType> arm_compute::IContext *create_backend_ctx(const AclContextOptions *options) { - return new(std::nothrow) ContextType(options); + return new (std::nothrow) ContextType(options); } bool is_target_valid(AclTarget target) { - return arm_compute::utils::is_in(target, { AclCpu, AclGpuOcl }); + return arm_compute::utils::is_in(target, {AclCpu, AclGpuOcl}); } bool are_context_options_valid(const AclContextOptions *options) { ARM_COMPUTE_ASSERT_NOT_NULLPTR(options); - return arm_compute::utils::is_in(options->mode, { AclPreferFastRerun, AclPreferFastStart }); + return arm_compute::utils::is_in(options->mode, {AclPreferFastRerun, AclPreferFastStart}); } arm_compute::IContext *create_context(AclTarget target, const AclContextOptions *options) { - switch(target) + ARM_COMPUTE_UNUSED(options); + + switch (target) { #ifdef ARM_COMPUTE_CPU_ENABLED case AclCpu: @@ -73,29 +76,27 @@ arm_compute::IContext *create_context(AclTarget target, const AclContextOptions } } // namespace -extern "C" AclStatus AclCreateContext(AclContext *ctx, - AclTarget target, - const AclContextOptions *options) +extern "C" AclStatus AclCreateContext(AclContext *external_ctx, AclTarget target, const AclContextOptions *options) { - if(!is_target_valid(target)) + if (!is_target_valid(target)) { - ARM_COMPUTE_LOG_ERROR_ACL("Target is invalid"); + ARM_COMPUTE_LOG_ERROR_WITH_FUNCNAME_ACL("Target is invalid!"); return AclUnsupportedTarget; } - if(options != nullptr && !are_context_options_valid(options)) + if (options != nullptr && !are_context_options_valid(options)) { - ARM_COMPUTE_LOG_ERROR_ACL("Context options are invalid"); + ARM_COMPUTE_LOG_ERROR_WITH_FUNCNAME_ACL("Context options are invalid!"); return AclInvalidArgument; } - auto acl_ctx = create_context(target, options); - if(ctx == nullptr) + auto ctx = create_context(target, options); + if (ctx == nullptr) { - ARM_COMPUTE_LOG_ERROR_ACL("Couldn't allocate internal resources for context creation"); + ARM_COMPUTE_LOG_ERROR_WITH_FUNCNAME_ACL("Couldn't allocate internal resources for context creation!"); return AclOutOfMemory; } - *ctx = acl_ctx; + *external_ctx = ctx; return AclSuccess; } @@ -106,13 +107,12 @@ extern "C" AclStatus AclDestroyContext(AclContext external_ctx) IContext *ctx = get_internal(external_ctx); - StatusCode status = StatusCode::Success; - status = detail::validate_internal_context(ctx); + StatusCode status = detail::validate_internal_context(ctx); ARM_COMPUTE_RETURN_CENUM_ON_FAILURE(status); - if(ctx->refcount() != 0) + if (ctx->refcount() != 0) { - ARM_COMPUTE_LOG_ERROR_ACL("Context has references on it that haven't been released"); + ARM_COMPUTE_LOG_ERROR_WITH_FUNCNAME_ACL("Context has references on it that haven't been released!"); // TODO: Fix the refcount with callback when reaches 0 } |