From 101de503901f503d8a6f741c3aaea359bc1aafb7 Mon Sep 17 00:00:00 2001 From: David Mansell Date: Tue, 6 Feb 2018 17:11:21 +0000 Subject: COMPMID-896: Replace legacy 4x4 u8 GEMM kernel with safe version. It's not safe to accumulate two u8xu8 results into a u16 accumulator. This changes the kernel to use uadalp after every single multiply. Correct the test fixture as well. Change-Id: I011b90033c4673e55b843d079e3f7d185b1df330 Reviewed-on: https://eu-gerrit-1.euhpc.arm.com/119096 Tested-by: Jenkins Reviewed-by: Georgios Pinitas --- tests/validation/fixtures/GEMMLowpAssemblyFixture.h | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'tests/validation/fixtures/GEMMLowpAssemblyFixture.h') diff --git a/tests/validation/fixtures/GEMMLowpAssemblyFixture.h b/tests/validation/fixtures/GEMMLowpAssemblyFixture.h index ff33c9db39..d6b94a197d 100644 --- a/tests/validation/fixtures/GEMMLowpAssemblyFixture.h +++ b/tests/validation/fixtures/GEMMLowpAssemblyFixture.h @@ -1,5 +1,5 @@ /* - * Copyright (c) 2017 ARM Limited. + * Copyright (c) 2017-2018 ARM Limited. * * SPDX-License-Identifier: MIT * @@ -98,8 +98,8 @@ protected: } else { - fill(AccessorType(a), 0, 0, 128); - fill(AccessorType(b), 1, 0, 128); + fill(AccessorType(a), 0, 0, 255); + fill(AccessorType(b), 1, 0, 255); } fill(AccessorType(c), 2, 0, 0); @@ -124,8 +124,8 @@ protected: } else { - fill(a, 0, 0, 128); - fill(b, 1, 0, 128); + fill(a, 0, 0, 255); + fill(b, 1, 0, 255); } return reference::gemmlowp(a, b); -- cgit v1.2.1