| commit | a07d0b6b30c95f2d1f8fd687d3bffc7ec433a75d | |
| author | Noel Grandin <noel.grandin@collabora.co.uk> | |
| Thu, 22 Dec 2022 16:41:19 +0000 (22 18:41 +0200) | ||
| committer | Andras Timar <andras.timar@collabora.com> | |
| Mon, 23 Jan 2023 19:16:16 +0000 (23 20:16 +0100) | ||
| tree | 31615eb2bae1b84dde0db421755b29f5c74006d1 | treesnapshot (tar.gz zip) |
| parent | 88b65f6abf2600f4f423181f805aa9d21c381d18 | commitdiff |
optimise SUMPRODUCT(IF..) a little
Move the AddSub calculation inside ScMatrix so we
can use an iterator to walk the matrix and avoid lookup
cost for each element.
Shaves 50% off the time spent here in my test sheet.
Change-Id: I171d7dd4ae86419a563342a4120d14106e8d71db
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144826
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Move the AddSub calculation inside ScMatrix so we
can use an iterator to walk the matrix and avoid lookup
cost for each element.
Shaves 50% off the time spent here in my test sheet.
Change-Id: I171d7dd4ae86419a563342a4120d14106e8d71db
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144826
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
| sc/inc/scmatrix.hxx | diffblobblamehistory | |
| sc/source/core/tool/interpr5.cxx | diffblobblamehistory | |
| sc/source/core/tool/scmatrix.cxx | diffblobblamehistory |