[record_use] Add recorded uses to link hooks by dcharkes · Pull Request #184869 · flutter/flutter

dcharkes · 2026-04-10T06:40:27Z

Bug:

[record_use] Completeness: Flutter: Add experiment to Flutter dart-lang/native#2690

Invokes the Dart compiler with --recorded-uses and pipes the result to the link hook invocation if the Flutter experiment is enabled.

Testing:

Changes the existing integration test to expect the right number of translations (4 -> 2).
Enables the web integration test, as fixes have rolled from Dart to Flutter now.

Implementation quirks:

Accessing the experiment flag required updating a bunch of tests to have the feature flags accessible.

Does not yet:

Split the build and link hooks into separate targets. The build hooks can run in parallel with the Dart compilation, the link hooks can not. (So for now, when the experiment is enabled run the dart build hooks after dart compilation instead of concurrently.) I'll follow up with a separate CL to reshuffle the targets.

fluttergithubbot · 2026-04-10T08:31:37Z

An existing Git SHA, 3b3277ae1f1e5092cbcf9a02f2d8d08d5f8141ec, was detected, and no actions were taken.

To re-trigger presubmits after closing or re-opeing a PR, or pushing a HEAD commit (i.e. with --force) that already was pushed before, push a blank commit (git commit --allow-empty -m "Trigger Build") or rebase to continue.

reidbaker

I am not familiar with this work but the android changes are ok with me.

bkonyi

LGTM with one nit.

goderbauer

LGTM

fluttergithubbot · 2026-04-14T07:47:43Z

An existing Git SHA, d54a3600f0e7a91c3afb6a7441a38c09937c3103, was detected, and no actions were taken.

To re-trigger presubmits after closing or re-opeing a PR, or pushing a HEAD commit (i.e. with --force) that already was pushed before, push a blank commit (git commit --allow-empty -m "Trigger Build") or rebase to continue.

…er#184869)

flutter/flutter@2fa45e0...c1b14e9 2026-04-14 [email protected] Rebuild flutter tool skill (flutter/flutter#184975) 2026-04-14 [email protected] Roll Skia from 0851d988db03 to 4c382df6a25a (1 revision) (flutter/flutter#185025) 2026-04-14 [email protected] Roll Dart SDK from 5504504b38c2 to ee5afcef0596 (1 revision) (flutter/flutter#185024) 2026-04-14 [email protected] [ci] Split up integration.shard record_use_test.dart (flutter/flutter#185022) 2026-04-14 [email protected] Roll Skia from d34c84df4c37 to 0851d988db03 (3 revisions) (flutter/flutter#185012) 2026-04-14 [email protected] [record_use] Add recorded uses to link hooks (flutter/flutter#184869) 2026-04-14 [email protected] Roll Skia from 0e98a9c635bb to d34c84df4c37 (5 revisions) (flutter/flutter#185009) 2026-04-14 [email protected] Roll Dart SDK from ef28089d6533 to 5504504b38c2 (3 revisions) (flutter/flutter#185008) 2026-04-14 [email protected] Roll Fuchsia Linux SDK from K_2AkZL3Drs6cGE1q... to rB8LAuZL_DwHMssTU... (flutter/flutter#185007) 2026-04-14 [email protected] [fuchsia] Replace ambient-replace-as-executable with VmexResource. (flutter/flutter#184967) 2026-04-13 [email protected] [Impeller] Commands that don't specify their own viewports get the viewport of the render pass. (flutter/flutter#177473) 2026-04-13 [email protected] Roll Skia from bc1df263ff3f to 0e98a9c635bb (1 revision) (flutter/flutter#184995) 2026-04-13 [email protected] Update autosubmit guide with the emergency label (flutter/flutter#184993) 2026-04-13 [email protected] [flutter_tools] Cache pubspec reads and share PackageGraph/PackageConfig across workspace packages during pub get post-processing (flutter/flutter#184528) 2026-04-13 [email protected] Fix codesign verification test for SwiftPM Add to App (flutter/flutter#184980) 2026-04-13 [email protected] Preprovision Android NDK for flavored builds and reuse matching unflavored NDKs (flutter/flutter#183555) 2026-04-13 [email protected] Reland "Disable async mode with LLDB" (flutter/flutter#184970) 2026-04-13 [email protected] Roll Skia from 55ddd6bb8be5 to bc1df263ff3f (6 revisions) (flutter/flutter#184968) 2026-04-13 [email protected] Expose platform specific handles for multi-window API (flutter/flutter#184662) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC [email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Bug: * dart-lang/native#2690 Invokes the Dart compiler with `--recorded-uses` and pipes the result to the link hook invocation if the Flutter experiment is enabled. Testing: * Changes the existing integration test to expect the right number of translations (4 -> 2). * Enables the web integration test, as fixes have rolled from Dart to Flutter now. Implementation quirks: * Accessing the experiment flag required updating a bunch of tests to have the feature flags accessible. Does not yet: * Split the build and link hooks into separate targets. The build hooks can run in parallel with the Dart compilation, the link hooks can not. (So for now, when the experiment is enabled run the dart build hooks _after_ dart compilation instead of concurrently.) I'll follow up with a separate CL to reshuffle the targets.

…r#11506) flutter/flutter@2fa45e0...c1b14e9 2026-04-14 [email protected] Rebuild flutter tool skill (flutter/flutter#184975) 2026-04-14 [email protected] Roll Skia from 0851d988db03 to 4c382df6a25a (1 revision) (flutter/flutter#185025) 2026-04-14 [email protected] Roll Dart SDK from 5504504b38c2 to ee5afcef0596 (1 revision) (flutter/flutter#185024) 2026-04-14 [email protected] [ci] Split up integration.shard record_use_test.dart (flutter/flutter#185022) 2026-04-14 [email protected] Roll Skia from d34c84df4c37 to 0851d988db03 (3 revisions) (flutter/flutter#185012) 2026-04-14 [email protected] [record_use] Add recorded uses to link hooks (flutter/flutter#184869) 2026-04-14 [email protected] Roll Skia from 0e98a9c635bb to d34c84df4c37 (5 revisions) (flutter/flutter#185009) 2026-04-14 [email protected] Roll Dart SDK from ef28089d6533 to 5504504b38c2 (3 revisions) (flutter/flutter#185008) 2026-04-14 [email protected] Roll Fuchsia Linux SDK from K_2AkZL3Drs6cGE1q... to rB8LAuZL_DwHMssTU... (flutter/flutter#185007) 2026-04-14 [email protected] [fuchsia] Replace ambient-replace-as-executable with VmexResource. (flutter/flutter#184967) 2026-04-13 [email protected] [Impeller] Commands that don't specify their own viewports get the viewport of the render pass. (flutter/flutter#177473) 2026-04-13 [email protected] Roll Skia from bc1df263ff3f to 0e98a9c635bb (1 revision) (flutter/flutter#184995) 2026-04-13 [email protected] Update autosubmit guide with the emergency label (flutter/flutter#184993) 2026-04-13 [email protected] [flutter_tools] Cache pubspec reads and share PackageGraph/PackageConfig across workspace packages during pub get post-processing (flutter/flutter#184528) 2026-04-13 [email protected] Fix codesign verification test for SwiftPM Add to App (flutter/flutter#184980) 2026-04-13 [email protected] Preprovision Android NDK for flavored builds and reuse matching unflavored NDKs (flutter/flutter#183555) 2026-04-13 [email protected] Reland "Disable async mode with LLDB" (flutter/flutter#184970) 2026-04-13 [email protected] Roll Skia from 55ddd6bb8be5 to bc1df263ff3f (6 revisions) (flutter/flutter#184968) 2026-04-13 [email protected] Expose platform specific handles for multi-window API (flutter/flutter#184662) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC [email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

github-actions Bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Apr 10, 2026

dcharkes force-pushed the record-use-implementation branch from a7e7dad to 34b91f3 Compare April 10, 2026 07:12

dcharkes added the CICD Run CI/CD label Apr 10, 2026

dcharkes mentioned this pull request Apr 10, 2026

[hooks_runner] BuildResult serialization dart-lang/native#3300

Merged

dcharkes force-pushed the record-use-implementation branch from 34b91f3 to cae5395 Compare April 10, 2026 07:28

github-actions Bot removed the CICD Run CI/CD label Apr 10, 2026

dcharkes added the CICD Run CI/CD label Apr 10, 2026

dcharkes force-pushed the record-use-implementation branch from cae5395 to 15ec6b3 Compare April 10, 2026 07:48

github-actions Bot removed the CICD Run CI/CD label Apr 10, 2026

dcharkes added the CICD Run CI/CD label Apr 10, 2026

dcharkes force-pushed the record-use-implementation branch from 15ec6b3 to 3b3277a Compare April 10, 2026 08:30

github-actions Bot removed the CICD Run CI/CD label Apr 10, 2026

dcharkes added CICD Run CI/CD labels Apr 10, 2026

dcharkes force-pushed the record-use-implementation branch from 3b3277a to 2347966 Compare April 10, 2026 08:34

dcharkes added CICD Run CI/CD and removed CICD Run CI/CD labels Apr 10, 2026

dcharkes force-pushed the record-use-implementation branch from 2347966 to bb34e07 Compare April 10, 2026 08:40

dcharkes added CICD Run CI/CD and removed CICD Run CI/CD labels Apr 10, 2026

[record_use] Add recorded uses to link hooks

0af70a5

dcharkes force-pushed the record-use-implementation branch from bb34e07 to 0af70a5 Compare April 10, 2026 08:47

dcharkes added CICD Run CI/CD and removed CICD Run CI/CD labels Apr 10, 2026

github-actions Bot added a: desktop Running on desktop team-ios Owned by iOS platform team team-macos Owned by the macOS platform team team-windows Owned by the Windows platform team and removed CICD Run CI/CD labels Apr 10, 2026

reidbaker previously approved these changes Apr 10, 2026

View reviewed changes

dcharkes mentioned this pull request Apr 10, 2026

[record_use] Run build hooks and link hooks in separate targets #184880

Merged

bkonyi previously approved these changes Apr 10, 2026

View reviewed changes

Comment thread packages/flutter_tools/lib/src/build_system/targets/native_assets.dart Outdated

address comment

d54a360

dcharkes dismissed stale reviews from bkonyi and reidbaker via d54a360 April 12, 2026 14:26

github-actions Bot removed the CICD Run CI/CD label Apr 12, 2026

dcharkes added the CICD Run CI/CD label Apr 12, 2026

goderbauer approved these changes Apr 13, 2026

View reviewed changes

bkonyi approved these changes Apr 13, 2026

View reviewed changes

dcharkes added CICD Run CI/CD autosubmit Merge PR when tree becomes green via auto submit App and removed CICD Run CI/CD labels Apr 14, 2026

Trigger Build

5885630

github-actions Bot removed the CICD Run CI/CD label Apr 14, 2026

dcharkes added the CICD Run CI/CD label Apr 14, 2026

auto-submit Bot added this pull request to the merge queue Apr 14, 2026

Merged via the queue into flutter:master with commit c4391ec Apr 14, 2026
154 checks passed

flutter-dashboard Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Apr 14, 2026

engine-flutter-autoroll mentioned this pull request Apr 14, 2026

Roll Flutter from 2fa45e01aa5c to 252d635a01de (18 revisions) flutter/packages#11503

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 14, 2026

c4391ec85 [record_use] Add recorded uses to link hooks (flutter/flutt…

d28eab1

…er#184869)

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 14, 2026

c4391ec85 [record_use] Add recorded uses to link hooks (flutter/flutt…

b429833

…er#184869)

engine-flutter-autoroll mentioned this pull request Apr 14, 2026

Roll Flutter from 2fa45e01aa5c to c1b14e92dcfb (19 revisions) flutter/packages#11506

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[record_use] Add recorded uses to link hooks#184869

[record_use] Add recorded uses to link hooks#184869
auto-submit[bot] merged 5 commits into
flutter:masterfrom
dcharkes:record-use-implementation

dcharkes commented Apr 10, 2026 •

edited

Loading

Uh oh!

fluttergithubbot commented Apr 10, 2026

Uh oh!

reidbaker left a comment

Uh oh!

bkonyi left a comment

Uh oh!

Uh oh!

goderbauer left a comment

Uh oh!

fluttergithubbot commented Apr 14, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Uh oh!

Conversation

dcharkes commented Apr 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

fluttergithubbot commented Apr 10, 2026

Uh oh!

reidbaker left a comment

Choose a reason for hiding this comment

Uh oh!

bkonyi left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

goderbauer left a comment

Choose a reason for hiding this comment

Uh oh!

fluttergithubbot commented Apr 14, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

dcharkes commented Apr 10, 2026 •

edited

Loading