Skip to content

git: More performance improvements when toggling between diff views#49400

Merged
cole-miller merged 10 commits intomainfrom
split-view-perf
Feb 17, 2026
Merged

git: More performance improvements when toggling between diff views#49400
cole-miller merged 10 commits intomainfrom
split-view-perf

Conversation

@cole-miller
Copy link
Member

@cole-miller cole-miller commented Feb 17, 2026

  • Defer syncing block maps from set_companion, eliminating some redundant recomputations
  • Emit one large multibuffer edit from set_show_deleted_hunks instead of many small edits, to avoid bad case for block map

This cuts hangs roughly in half when toggling between views in a large diff (1000 commits from the chromium repository).

Release Notes:

  • Improved performance with large diffs when toggling between diff views.

kubkon and others added 8 commits February 17, 2026 15:06
Co-authored-by: Lukas Wirth <lukas@zed.dev>
Co-authored-by: Cole Miller <cole@zed.dev>
Co-authored-by: Lukas Wirth <lukas@zed.dev>
Co-authored-by: Jakub Konka <kubkon@jakubkonka.com>
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Feb 17, 2026
@zed-community-bot zed-community-bot bot added the staff Pull requests authored by a current member of Zed staff label Feb 17, 2026
cole-miller and others added 2 commits February 17, 2026 15:40
Co-authored-by: Jakub Konka <kubkon@jakubkonka.com>
@cole-miller cole-miller enabled auto-merge (squash) February 17, 2026 22:04
@cole-miller cole-miller merged commit e8816f2 into main Feb 17, 2026
27 checks passed
@cole-miller cole-miller deleted the split-view-perf branch February 17, 2026 22:09
@cole-miller
Copy link
Member Author

/cherry-pick preview

github-actions bot pushed a commit that referenced this pull request Feb 17, 2026
…49400)

- Defer syncing block maps from `set_companion`, eliminating some
redundant recomputations
- Emit one large multibuffer edit from `set_show_deleted_hunks` instead
of many small edits, to avoid bad case for block map

This cuts hangs roughly in half when toggling between views in a large
diff (1000 commits from the chromium repository).

<!-- Before you mark this PR as ready for review, make sure that you
have:
- [ ] Added a solid test coverage and/or screenshots from doing manual
testing
- [ ] Done a self-review taking into account security and performance
aspects
- [ ] Aligned any UI changes with the [UI
checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)
-->

Release Notes:

- Improved performance with large diffs when toggling between diff
views.

---------

Co-authored-by: Jakub Konka <kubkon@jakubkonka.com>
Co-authored-by: Lukas Wirth <lukas@zed.dev>
zed-zippy bot added a commit that referenced this pull request Feb 17, 2026
…49400) (cherry-pick to preview) (#49412)

Cherry-pick of #49400 to preview

----
- Defer syncing block maps from `set_companion`, eliminating some
redundant recomputations
- Emit one large multibuffer edit from `set_show_deleted_hunks` instead
of many small edits, to avoid bad case for block map

This cuts hangs roughly in half when toggling between views in a large
diff (1000 commits from the chromium repository).

<!-- Before you mark this PR as ready for review, make sure that you
have:
- [ ] Added a solid test coverage and/or screenshots from doing manual
testing
- [ ] Done a self-review taking into account security and performance
aspects
- [ ] Aligned any UI changes with the [UI

checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)
-->

Release Notes:

- Improved performance with large diffs when toggling between diff
views.

---------

Co-authored-by: Jakub Konka <kubkon@jakubkonka.com>
Co-authored-by: Lukas Wirth <lukas@zed.dev>

Co-authored-by: Cole Miller <cole@zed.dev>
Co-authored-by: Jakub Konka <kubkon@jakubkonka.com>
Co-authored-by: Lukas Wirth <lukas@zed.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement staff Pull requests authored by a current member of Zed staff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants