Skip to content

feat: add DD table files to supplement file handling#3671

Draft
bdero wants to merge 2 commits intomainfrom
bdero/cascade-limiter
Draft

feat: add DD table files to supplement file handling#3671
bdero wants to merge 2 commits intomainfrom
bdero/cascade-limiter

Conversation

@bdero
Copy link
Copy Markdown
Member

@bdero bdero commented Mar 31, 2026

Summary

  • Adds App.dd.link and App.dd_callers.link to the supplement file list so they get copied between snapshot directories during patch builds.
  • These files are produced by the 2-pass release build (in Flutter tooling) and consumed by aot_tools during patch linking for DD slot resolution via winner voting.

Test plan

  • Verify DD table files are copied during patch builds
  • Run existing shorebird CLI tests

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 31, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@bdero bdero force-pushed the bdero/cascade-limiter branch from 4ba3c79 to 7777015 Compare March 31, 2026 22:20
@bdero bdero changed the title WIP: Add DD table files to supplement file handling feat: add DD table files to supplement file handling Mar 31, 2026
bdero added 2 commits April 3, 2026 12:57
Include App.dd.link and App.dd_callers.link in the list of supplement
files that are copied between snapshot directories during patch builds.
These files are produced by the 2-pass release build and used by the
linker's winner voting algorithm to resolve DD slot mappings.
Allows overriding the Dynamic Dispatch table cascade byte threshold
when creating a release. Passed to Flutter tools via the
SHOREBIRD_DD_MAX_BYTES environment variable for backwards
compatibility: older Flutter builds that don't recognize the variable
will silently ignore it.

Usage: shorebird release ios --dd-max-bytes=5000
@bdero bdero force-pushed the bdero/cascade-limiter branch from cc39a11 to 7e14b32 Compare April 3, 2026 19:58
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