https://bugs.documentfoundation.org/show_bug.cgi?id=109097

--- Comment #4 from Markus Mohrhard <markus.mohrh...@googlemail.com> ---
(In reply to Aron Budea from comment #3)
> Created attachment 134615 [details]
> Callgrind trace
> 
> Here's a Callgrind output (well, the first of the 8 files). Unfortunately
> the spreadsheet involved cannot be shared.

Well, that one shows that the performance problem is in the matrix code where
we use index based access to get each value out of two matrices and insert into
a new one. We have existing code to do this with iterators which will improve
the performance of that code significantly.

At least from a quick look I don't see anything in this file that reflects the
ScColumn::GetCellValue problem Michael mentioned. Based on a quick reading 75%
of the import time is spend in 12 lcl_GetMinExtend calls with all the time
spend in inefficient iteration through the matrix.

Maybe you have a different document showing real problems but I doubt that this
one corresponds to the problems mentioned by Michael. This one is actually a
quite "easy" fix if you follow the existing ScMatrix performance hacks.

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

Reply via email to