Skip to content

Commit 6002964

Browse files
jensjohaCommit Queue
authored and
Commit Queue
committed
[analyzer] Train snapshot more than --help
The analyzer snapshot is currently "trained" with `--help`, i.e. not really trained at all. Doing some archaeology, it actually used to be trained, but it was disabled in https://dart-review.git.corp.google.com/c/sdk/+/77228 because of Fuchsia issues (#34616). Although the troublesome stuff was perhaps fixed in https://dart-review.git.corp.google.com/c/sdk/+/77422 it was seemingly never re-enabled. That was 2018. This CL re-enables it, taking an initial uncached analysis of dart2js to ~51% (~9.5s vs ~18.6s), of the analyzer to ~75% (~22.2s vs ~29.6) and of the front_end to ~50% (~7.7s vs ~15.2s). This won’t change the peak performance of the analyzer, but will certainly - as seen above - make it faster when first loaded. Change-Id: I613e6a5b52d00da15948cf936f20ea36ad28818e Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/269683 Reviewed-by: Ryan Macnak <[email protected]> Commit-Queue: Jens Johansen <[email protected]>
1 parent 39c6831 commit 6002964

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

utils/analysis_server/BUILD.gn

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,7 @@ import("../application_snapshot.gni")
77
application_snapshot("analysis_server") {
88
main_dart = "../../pkg/analysis_server/bin/server.dart"
99
training_args = [
10-
"--help",
11-
# TODO(34616): This is broken on Fuchsia.
12-
# "--sdk=" + rebase_path("../../sdk/"),
13-
# "--train-using=" + rebase_path("../../pkg/analyzer_cli")
10+
"--sdk=" + rebase_path("../../sdk/"),
11+
"--train-using=" + rebase_path("../../pkg/compiler/lib"),
1412
]
1513
}

0 commit comments

Comments
 (0)