Skip to content

Conversation

@wouterlucas
Copy link
Contributor

Work in progress but this PR does:

  • Reduce branching in hot paths (or attempt to clean up)
  • Reduce look ups in variables, const them out earlier
  • Reduce assert fn calls where we no longer need them
  • Deduplicate a lot of code

Avoiding Text paths for now, as that's part of another change.

Big gains on node creation / node deletion, still investigating the update/swap cases.

@wouterlucas wouterlucas force-pushed the performance/micro-optimisations branch from 7a1c89c to 8a23805 Compare May 14, 2025 17:29
@wouterlucas wouterlucas marked this pull request as ready for review May 21, 2025 10:08
@wouterlucas wouterlucas changed the title [WIP] Micro optimisations & perf updates Micro optimisations & perf updates May 21, 2025
@wouterlucas wouterlucas added this pull request to the merge queue May 23, 2025
Merged via the queue into main with commit a3106f0 May 23, 2025
2 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.

3 participants