Skip to content

Commit ce6634e

Browse files
rmacnak-googleCommit Queue
authored andcommitted
[build] Collapse now-identical application_snapshot and kernel_application_snapshot.
Change-Id: I6aabe13b9e4adcfcfb073197e9b6b5605e16e30c Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/310968 Reviewed-by: Alexander Aprelev <[email protected]> Commit-Queue: Ryan Macnak <[email protected]>
1 parent 1f953c3 commit ce6634e

File tree

2 files changed

+2
-67
lines changed

2 files changed

+2
-67
lines changed

utils/application_snapshot.gni

Lines changed: 1 addition & 66 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ declare_args() {
4545
#
4646
# output (optional):
4747
# Overrides the full output path.
48-
template("_application_snapshot") {
48+
template("application_snapshot") {
4949
assert(defined(invoker.main_dart), "Must specify 'main_dart'")
5050
assert(defined(invoker.training_args), "Must specify 'training_args'")
5151
if (defined(invoker.dart_snapshot_kind)) {
@@ -184,68 +184,3 @@ template("_application_snapshot") {
184184
}
185185
}
186186
}
187-
188-
# Creates an app-jit snapshot for a Dart2 program based on a training run.
189-
#
190-
# Parameters:
191-
# main_dart (required):
192-
# The entrypoint to the Dart application.
193-
#
194-
# training_args (required):
195-
# Arguments to pass to the Dart application for the training run.
196-
#
197-
# vm_args (optional):
198-
# Additional arguments to the Dart VM.
199-
#
200-
# name (optional):
201-
# The name of the snapshot if different from the target name. The output
202-
# will be in $root_gen_dir/$name.dart.snapshot.
203-
#
204-
# deps (optional):
205-
# Any build dependencies.
206-
#
207-
# dot_packages (optional):
208-
# The package config file for the app. Defaults to
209-
# $_dart_root/.dart_tool/package_config.json.
210-
#
211-
# output (optional):
212-
# Overrides the full output path.
213-
template("application_snapshot") {
214-
_application_snapshot(target_name) {
215-
forward_variables_from(invoker, "*")
216-
if (!defined(invoker.deps)) {
217-
deps = []
218-
}
219-
}
220-
}
221-
222-
# Creates an app-jit snapshot for the common FE based on a training run.
223-
#
224-
# Parameters:
225-
# main_dart (required):
226-
# The entrypoint to the Dart application.
227-
#
228-
# training_args (required):
229-
# Arguments to pass to the Dart application for the training run.
230-
#
231-
# vm_args (optional):
232-
# Additional arguments to the Dart VM.
233-
#
234-
# name (optional):
235-
# The name of the snapshot if different from the target name. The output
236-
# will be in $root_gen_dir/$name.dart.snapshot.
237-
#
238-
# deps (optional):
239-
# Any build dependencies.
240-
#
241-
# dot_packages (optional):
242-
# The packages config file for the app. Defaults to
243-
# $_dart_root/.dart_tool/package_config.json.
244-
#
245-
# output (optional):
246-
# Overrides the full output path.
247-
template("kernel_application_snapshot") {
248-
_application_snapshot(target_name) {
249-
forward_variables_from(invoker, "*")
250-
}
251-
}

utils/kernel-service/BUILD.gn

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ group("kernel-service") {
1616
}
1717
}
1818

19-
kernel_application_snapshot("kernel-service_snapshot") {
19+
application_snapshot("kernel-service_snapshot") {
2020
main_dart = "../../pkg/vm/bin/kernel_service.dart"
2121
training_args = [
2222
"--train",

0 commit comments

Comments
 (0)