Skip to content

Reduce formatting allocation in CodeAnalysis process in scrolling speedometer#75912

Merged
ToddGrun merged 1 commit intodotnet:mainfrom
ToddGrun:dev/toddgrun/reduceFormattingAllocationInCodeAnalysisProcess
Nov 14, 2024
Merged

Reduce formatting allocation in CodeAnalysis process in scrolling speedometer#75912
ToddGrun merged 1 commit intodotnet:mainfrom
ToddGrun:dev/toddgrun/reduceFormattingAllocationInCodeAnalysisProcess

Conversation

@ToddGrun
Copy link
Copy Markdown
Contributor

TokenPairWithOperations[] accounts for 2.4% of total allocations in the test. Locally reproducing this demonstrated that there is usually a small number of tokens that are added, but occasionally a much larger token count is added. Doing a single shot allocation in the case where we are significantly increasing the SegmentedList size is more efficient than growing organically by adding single items.

image

…edometer

TokenPairWithOperations[] accounts for 2.4% of total allocations in the test. Locally reproducing this demonstrated that there is usually a small number of tokens that are added, but occasionally a much larger token count is added. Doing a single shot allocation in the case where we are significantly increasing the SegmentedList size is more efficient than growing organically by adding single items.
@ToddGrun ToddGrun requested a review from a team as a code owner November 14, 2024 03:58
@ghost ghost added Area-IDE untriaged Issues and PRs which have not yet been triaged by a lead labels Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-IDE untriaged Issues and PRs which have not yet been triaged by a lead VSCode

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants