X Tutup
Skip to content

Betsy: Generate BC1 table cache during compression#109725

Merged
Repiteo merged 1 commit intogodotengine:masterfrom
BlueCube3310:betsy-rt-bc1-cache
Oct 21, 2025
Merged

Betsy: Generate BC1 table cache during compression#109725
Repiteo merged 1 commit intogodotengine:masterfrom
BlueCube3310:betsy-rt-bc1-cache

Conversation

@BlueCube3310
Copy link
Contributor

Since caching is now implemented in Betsy, the encoding tables can be generated at compression time (like the upstream Betsy repository) without any performance loss. This decreases the amount of statically-allocated data in the executable.

@BlueCube3310 BlueCube3310 requested a review from a team as a code owner August 18, 2025 09:51
@BlueCube3310 BlueCube3310 added this to the 4.x milestone Aug 18, 2025
@fire fire moved this to Ready for review in Asset Pipeline Issue Triage Sep 21, 2025
@BlueCube3310 BlueCube3310 changed the title Betsy: Generate BC1 tables only when required Betsy: Generate BC1 table cache during compression Sep 23, 2025
Copy link
Member

@Calinou Calinou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested locally, it works as expected. Code looks good to me.

Linux x86_64 stripped + optimized editor binary size:

  • Before: 138,259,016 bytes
  • After: 138,256,584 bytes (-3 kB)

@BlueCube3310 BlueCube3310 force-pushed the betsy-rt-bc1-cache branch 2 times, most recently from 99272fd to 6804fdd Compare September 25, 2025 09:50
@BlueCube3310 BlueCube3310 modified the milestones: 4.x, 4.6 Sep 25, 2025
@Repiteo Repiteo merged commit 331d57d into godotengine:master Oct 21, 2025
20 checks passed
@github-project-automation github-project-automation bot moved this from Ready for review to Done in Asset Pipeline Issue Triage Oct 21, 2025
@Repiteo
Copy link
Contributor

Repiteo commented Oct 21, 2025

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

4 participants

X Tutup