commit 55523e53df6f5ec1e9dad382d370cc8fd7262aeb
author Jens Johansen <jensj@google.com> Thu Jul 10 00:02:13 2025 -0700
committer Commit Queue <dart-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu Jul 10 00:02:13 2025 -0700
tree d5495abd22acf9c1576d3171719760f06d52c9a7
parent 97bc401163213973ed45d9775492b0e09e69ccf3

[CFE] Fix instrumenter so it works for aot

For a while it's been supported to do
`dart compile aot-snapshot <dillfile>`
meaning we can make the instrumentation thing run in aot too.

Except we couldn't because it crashed if you tried.

This CL sets up the dependency to the support library correctly (or at
least correctly enough) so that it now works.

Verified by doing

```
$ out/ReleaseX64/dart pkg/front_end/tool/flame/instrumenter.dart pkg/front_end/tool/compile.dart --count
$ out/ReleaseX64/dart-sdk/bin/dart compile aot-snapshot pkg/front_end/tool/compile.dart.dill.instrumented.dill
$ out/ReleaseX64/dart-sdk/bin/dartaotruntime pkg/front_end/tool/compile.dart.dill.instrumented.aot pkg/front_end/tool/compile.dart
```

Change-Id: I1d9b526b6ba2fed1ef81599fc9399041cc8b6522
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/439520
Commit-Queue: Jens Johansen <jensj@google.com>
Reviewed-by: Chloe Stefantsova <cstefantsova@google.com>

pkg/front_end/tool/flame/instrumenter.dart

diff --git a/pkg/front_end/tool/flame/instrumenter.dart b/pkg/front_end/tool/flame/instrumenter.dart
index a205cbd..b0992d1 100644
--- a/pkg/front_end/tool/flame/instrumenter.dart
+++ b/pkg/front_end/tool/flame/instrumenter.dart
@@ -432,6 +432,8 @@
     List<Procedure> procedures,
     List<Constructor> constructors,
     Procedure instrumenterReport) {
+  mainProcedure.enclosingLibrary.dependencies
+      .add(new LibraryDependency.import(initializeProcedure.enclosingLibrary));
   Block block = new Block([
     new ExpressionStatement(new StaticInvocation(initializeProcedure,
         config.createBeforeArguments(procedures, constructors))),