From ce54b56e0d91a8e73f3ecfede6a2b2aa323aa1fd Mon Sep 17 00:00:00 2001 From: Georgios Pinitas Date: Thu, 14 Sep 2017 17:21:51 +0100 Subject: COMPMID-532: NEON ConvolutionLayer Quantised valgrind errors When matrix B was one dimensional the AccessWindowTranpose did not add bottom padding leading to invalid accesses. Switches the matrix B access window to AccessWindowStatic and allows AccessWindowStatic to add padding to 1D tensors. Change-Id: Ic7fbd20e0c85575b98a506c4c22d2f9ecd8995a9 Reviewed-on: http://mpd-gerrit.cambridge.arm.com/87757 Reviewed-by: Moritz Pflanzer Tested-by: Kaizen --- src/core/NEON/kernels/NEGEMMMatrixMultiplyKernel.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/core/NEON/kernels/NEGEMMMatrixMultiplyKernel.cpp') diff --git a/src/core/NEON/kernels/NEGEMMMatrixMultiplyKernel.cpp b/src/core/NEON/kernels/NEGEMMMatrixMultiplyKernel.cpp index b872bab928..d015ec3e41 100644 --- a/src/core/NEON/kernels/NEGEMMMatrixMultiplyKernel.cpp +++ b/src/core/NEON/kernels/NEGEMMMatrixMultiplyKernel.cpp @@ -1513,7 +1513,7 @@ void NEGEMMMatrixMultiplyKernel::configure(const ITensor *input0, const ITensor update_window_and_padding(win, AccessWindowRectangle(input0->info(), 0, 0, 4, 1, 1.f, 0.25f), - AccessWindowTranspose(input1->info(), 0, 0, 4, 1, 0.f, 0.25f), + AccessWindowStatic(input1->info(), 0, 0, input1->info()->tensor_shape().x(), ceil_to_multiple(input1->info()->tensor_shape().y(), 4)), output_access); output_access.set_valid_region(win, ValidRegion(Coordinates(0, 0), output->info()->tensor_shape())); -- cgit v1.2.1