diff options
| author | Anthony Barbier <anthony.barbier@arm.com> | 2018-06-04 14:09:09 +0100 |
|---|---|---|
| committer | Anthony Barbier <anthony.barbier@arm.com> | 2018-11-02 16:52:54 +0000 |
| commit | 7d920f4ce6e9b7b0f6d03fcac2c9598122de5404 (patch) | |
| tree | a4326027f67b4e8fd256e1047d35677552897e46 /src/runtime/CL/functions/CLGEMM.cpp | |
| parent | b95e2106d2e935cb8542f8b9600a2c95ee1c4a5d (diff) | |
| download | ComputeLibrary-7d920f4ce6e9b7b0f6d03fcac2c9598122de5404.tar.gz | |
COMPMID-1246 Fixed Window::scale()
- There used to be two problems with scale():
- The first argument of ceil_to_multiple was promoted to float
which broke the function (For example ceil_to_multiple( 24.0, 8 ) will return 31.
- "End - start" needs to be a multiple of step, not "end" (e.g start=1, end =5, step =4 is a valid dimension)
The reason it didn't break before is because Window::scale() was only
used on windows used by iterators, and therefore the "end" value is not
used in that context.
Change-Id: I1798db73014294ac82eed53c74eec3d4b8cb7d59
Reviewed-on: https://eu-gerrit-1.euhpc.arm.com/133967
Tested-by: Jenkins <bsgcomp@arm.com>
Reviewed-by: Pablo Tello <pablo.tello@arm.com>
Diffstat (limited to 'src/runtime/CL/functions/CLGEMM.cpp')
0 files changed, 0 insertions, 0 deletions