From 841c3e9f68b1954828a9137d6780b6d021354eb7 Mon Sep 17 00:00:00 2001 From: Sheri Zhang Date: Mon, 18 Oct 2021 10:04:12 +0100 Subject: Fix unused variable issue Resovles: COMPMID-4851 Signed-off-by: Sheri Zhang Change-Id: Ic79dadb17efb6ec5a08dbad02e17c0eaff48569f Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/6451 Tested-by: Arm Jenkins Reviewed-by: Pablo Marquez Tello Comments-Addressed: Arm Jenkins --- tests/validation/NEON/DirectConvolution3D.cpp | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/tests/validation/NEON/DirectConvolution3D.cpp b/tests/validation/NEON/DirectConvolution3D.cpp index ff40cf5b74..37c901d2ac 100644 --- a/tests/validation/NEON/DirectConvolution3D.cpp +++ b/tests/validation/NEON/DirectConvolution3D.cpp @@ -50,17 +50,6 @@ constexpr float tolerance_num = 0.07f; #endif /* __ARM_FEATURE_FP16_VECTOR_ARITHMETIC */ constexpr AbsoluteTolerance tolerance_fp32(0.001f); /**< Tolerance for floating point tests */ -/* The following tests are from real use-case that made DirectConvolution - * overflows in terms of its tensor indexing. This test case is using - * a separate tolerance due to the following reason. - * - It has shown that it requires generally larger absolute tolerance - * for large numbers or larger relative tolerance for small numbers. - * - With the first reason, since it is mainly testing index overflow, - * a value with a margin is used to avoid uninteded test failures - * during nightly. - */ -constexpr AbsoluteTolerance usecase_tolerance_fp32(0.05f); - /** Activation function Dataset*/ const auto ActivationFunctionsDataset = framework::dataset::make("ActivationInfo", { -- cgit v1.2.1