Skip to content
Snippets Groups Projects
Commit bd2384f5 authored by Milian Wolff's avatar Milian Wolff Committed by Stephen Kelly
Browse files

Optimize cmMakefile::ExpandVariablesInStringNew.

We can remove the temporary allocations required for the
default-constructed t_lookup passed into the openstack by refactoring
the code slightly. Furthermore, we use a vector instead of a stack,
since the latter is based on a deque which is not required for a
heap / lifo structure.

This patch removes ~215k allocations.

This hotspot was found with heaptrack.
parent ad9394f4
No related branches found
No related tags found
No related merge requests found
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment