Skip to content

Revert PR #96 GC startup changes#155

Merged
SudoPlz merged 1 commit intodiscord:0.81.4-discordfrom
SudoPlz:sudoplz/revert-pr96-gc
Mar 4, 2026
Merged

Revert PR #96 GC startup changes#155
SudoPlz merged 1 commit intodiscord:0.81.4-discordfrom
SudoPlz:sudoplz/revert-pr96-gc

Conversation

@SudoPlz
Copy link

@SudoPlz SudoPlz commented Mar 4, 2026

This reverts the "Avoid GC during the app start" change (originally from #96) on 0.81.4-discord by reverting commit 491af54.

The reason we're doing that was because despite our initial tests, where we tested the benefits of this optimization (which is TTI improvement) - were memory did not increase, it turns out in current tests that memory does regress a bit.

Specifically on my device tests I noticed a 48MB regression in memory 30s after app start and 137ms TTI improvement

This is not to be discarded for ever. We'll dig deeper into this, figure out why the regression is happening and find a way to only grab the benefit (TTI improvement) out of this without regressions.

@SudoPlz SudoPlz merged commit cef99b5 into discord:0.81.4-discord Mar 4, 2026
105 of 131 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant