Add benchmark for ELF "str2symtab" creation #943
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add a benchmark for the creation of our ELF "str2symtab". This creation should trigger a good chunk of our data structure traversing and "parsing" paths. The performance is roughly as follows:
The main reason for the addition of the benchmark is to get an idea how much performance we lost due to recent ELF parser changes: both the introduction of 32 bit support as well as the logic for working with different backends is assumed to have some runtime impact. Backporting the benchmark on top of commit 4688aa0 ("cli: Rework tracing output") results in the following performance numbers before said ELF changes:
The difference is reproducible, but arguably small and acceptable. We may be able to speed things up a bit more via the usage of smaller indexes.