diff options
author | David Mansell <David.Mansell@arm.com> | 2018-02-06 17:11:21 +0000 |
---|---|---|
committer | Anthony Barbier <anthony.barbier@arm.com> | 2018-11-02 16:45:42 +0000 |
commit | 101de503901f503d8a6f741c3aaea359bc1aafb7 (patch) | |
tree | 22f3268ea1c57a4a60af7e2cafe3b8a9af1d1b28 /tests | |
parent | 679463af405e72cbaf1ce41ef393fbe0db908e54 (diff) | |
download | ComputeLibrary-101de503901f503d8a6f741c3aaea359bc1aafb7.tar.gz |
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 <bsgcomp@arm.com>
Reviewed-by: Georgios Pinitas <georgios.pinitas@arm.com>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/validation/fixtures/GEMMLowpAssemblyFixture.h | 10 |
1 files changed, 5 insertions, 5 deletions
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<int32_t, T2>(a, b); |