D12897: Reserve space for the cachedLineForRanges Qhash (optimization)
Jaime Torres Amate
noreply at phabricator.kde.org
Tue May 15 15:47:16 UTC 2018
jtamate created this revision.
jtamate added reviewers: Kate, Frameworks.
Restricted Application added projects: Kate, Frameworks.
Restricted Application added subscribers: kde-frameworks-devel, kwrite-devel.
jtamate requested review of this revision.
REVISION SUMMARY
Most of the time was spent allocating space for the hashdata.
The perf numbers speak by themself:
from 42.3% (cycles inc.) F5849664: kwrite_reserver_perf_dwarf_before.png <https://phabricator.kde.org/F5849664>
to 16.1% (cycles inc.) F5849665: kwrite_reserver_perf_dwarf_after.png <https://phabricator.kde.org/F5849665>
TEST PLAN
repeat {
Paste, go to beginning of file
} 5 times
Undo 5 times
REPOSITORY
R39 KTextEditor
REVISION DETAIL
https://phabricator.kde.org/D12897
AFFECTED FILES
src/buffer/katetextblock.cpp
To: jtamate, #kate, #frameworks
Cc: kwrite-devel, kde-frameworks-devel, michaelh, kevinapavew, ngraham, bruns, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20180515/ad15755d/attachment.html>
More information about the KWrite-Devel
mailing list