]> git.ipfire.org Git - thirdparty/git.git/commit
strmap: enable faster clearing and reusing of strmaps
authorElijah Newren <newren@gmail.com>
Thu, 5 Nov 2020 00:22:40 +0000 (00:22 +0000)
committerJunio C Hamano <gitster@pobox.com>
Thu, 5 Nov 2020 01:09:49 +0000 (17:09 -0800)
commit6ccdfc2a206d8266ad7613999a0d3f6acdf44c89
treebd38936b27b908a589f18c47cc28ceaa9d4c10f2
parentb70c82e6edd33aa03afecd52b0265cf1d9762e1b
strmap: enable faster clearing and reusing of strmaps

When strmaps are used heavily, such as is done by my new merge-ort
algorithm, and strmaps need to be cleared but then re-used (because of
e.g. picking multiple commits to cherry-pick, or due to a recursive
merge having several different merges while recursing), free-ing and
reallocating map->table repeatedly can add up in time, especially since
it will likely be reallocated to a much smaller size but the previous
merge provides a good guide to the right size to use for the next merge.

Introduce strmap_partial_clear() to take advantage of this type of
situation; it will act similar to strmap_clear() except that
map->table's entries are zeroed instead of map->table being free'd.
Making use of this function reduced the cost of
clear_or_reinit_internal_opts() by about 20% in mert-ort, and dropped
the overall runtime of my rebase testcase by just under 2%.

Signed-off-by: Elijah Newren <newren@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
strmap.c
strmap.h