Revert ""[SDK/VM] - Rename dart_precompiled_runtime to dartaotruntime, ensures we have a uniform name for the executable between the build directories and the SDK directory""

This reverts commit f81a402.

Reason for revert: golem benchmarks are failing to run

TEST=ci

Original change's description:
> "[SDK/VM] - Rename dart_precompiled_runtime to dartaotruntime, ensures we have a uniform name for the executable between the build directories and the SDK directory"
>
> Fixed golem breakage by temporarily copying dartaotruntime to dart_precompiled_runtime
>
> This reverts commit 75e6a74.
>
> TEST=ci
>
> Original change's description:
> > Revert "[SDK/VM] - Rename dart_precompiled_runtime to dartaotruntime, ensures we have a uniform name for the executable between the build directories and the SDK directory"
> >
> > This reverts commit 1b331d0.
> >
> > Reason for revert: golem builds are failing
> >
> > Original change's description:
> > > [SDK/VM] - Rename dart_precompiled_runtime to dartaotruntime, ensures we have a uniform name for the executable between the build directories and the SDK directory
> > >
> > > TEST=ci
> > >
>
> Change-Id: Id0f383eabb496c06c0acebc639c8e3b056ba82d0
> Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/393781
> Commit-Queue: Siva Annamalai <[email protected]>
> Reviewed-by: Ryan Macnak <[email protected]>

Change-Id: Iec494940412aa31dbefdc5280e35ae99e8cecb26
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/393764
Reviewed-by: Siva Annamalai <[email protected]>
Reviewed-by: Liam Appelbe <[email protected]>
Bot-Commit: Rubber Stamper <[email protected]>
Commit-Queue: Siva Annamalai <[email protected]>

Author: a-siva

BUILD.gn

@@ -86,7 +86,7 @@ group("run_ffi_unit_tests") {
 group("runtime_precompiled") {
   import("runtime/runtime_args.gni")
   deps = [
-    "runtime/bin:dartaotruntime",
+    "runtime/bin:dart_precompiled_runtime",
     "runtime/bin:gen_snapshot",
     "runtime/bin:gen_snapshot($host_toolchain)",
     "runtime/bin:process_test",
@@ -110,15 +110,7 @@ group("create_platform_sdk") {
 }
 
 group("dart2js") {
-  import("runtime/runtime_args.gni")
-  if (dart_target_arch != "ia32" && dart_target_arch != "x86") {
-    deps = [
-      ":runtime_precompiled",
-      "utils/compiler:dart2js_sdk_aot",
-    ]
-  } else {
-    deps = [ "utils/compiler:dart2js" ]
-  }
+  deps = [ "utils/compiler:dart2js" ]
 }
 
 group("dart2wasm_platform") {
@@ -156,19 +148,7 @@ group("dartanalyzer") {
 }
 
 group("ddc") {
-  import("runtime/runtime_args.gni")
-  if (dart_target_arch != "ia32" && dart_target_arch != "x86") {
-    deps = [
-      ":runtime_precompiled",
-      "utils/bazel:kernel_worker_aot",
-      "utils/ddc:dartdevc_aot",
-    ]
-  } else {
-    deps = [
-      "utils/bazel:kernel_worker",
-      "utils/ddc:dartdevc",
-    ]
-  }
+  deps = [ "utils/ddc:dartdevc" ]
 }
 
 group("analysis_server") {
@@ -387,7 +367,7 @@ if (is_fuchsia) {
 
   test_binaries = [
     "dart",
-    "dartaotruntime",
+    "dart_precompiled_runtime",
     "run_vm_tests",
   ]
 

benchmarks/NativeCall/native/native_functions.c

@@ -7,7 +7,7 @@
 #include <string.h>
 
 // TODO(dartbug.com/40579): This requires static linking to either link
-// dart.exe or dartaotruntime.exe on Windows.
+// dart.exe or dart_precompiled_runtime.exe on Windows.
 // The sample currently fails on Windows in AOT mode.
 #include "include/dart_api.h"
 

docs/Kernel-developer-notes.md

@@ -28,7 +28,7 @@ Run end-to-end tests using dartk + VM:
 
 Optionally (this is slow) run end-to-end tests using AOT:
 ```
-./tools/build.py runtime_precompiled
+./tools/build.py dart_precompiled_runtime
 ./tools/test.py -cdartkp -rdart_precompiled language co19
 ```
 

pkg/dart2wasm/tool/compile_benchmark

@@ -35,7 +35,7 @@ BIN_DIR="$OUT_DIR/$DART_CONFIGURATION"
 
 BINARYEN="$BIN_DIR/wasm-opt"
 DART="$BIN_DIR/dart"
-DART_AOT_RUNTIME="$BIN_DIR/dartaotruntime"
+DART_AOT_RUNTIME="$BIN_DIR/dart_precompiled_runtime"
 LIBRARIES_JSON_ARG="--libraries-spec=$SDK_DIR/sdk/lib/libraries.json"
 
 function find_flags {

pkg/dartdev/lib/src/sdk.dart

@@ -54,8 +54,8 @@ class Sdk {
       ? path.absolute(
           sdkPath,
           Platform.isWindows
-              ? 'dartaotruntime_product.exe'
-              : 'dartaotruntime_product',
+              ? 'dart_precompiled_runtime_product.exe'
+              : 'dart_precompiled_runtime_product',
         )
       : path.absolute(
           sdkPath,

pkg/dev_compiler/test/worker/worker_test.dart

@@ -25,19 +25,17 @@ void main() {
   var dartAotRuntime = p.absolute(
       sdkPath,
       Platform.isWindows
-          ? 'dartaotruntime_product.exe'
-          : 'dartaotruntime_product');
-  var snapshotName = _resolvePath('gen/dartdevc_aot_product.dart.snapshot');
+          ? 'dart_precompiled_runtime_product.exe'
+          : 'dart_precompiled_runtime_product');
   if (!File(dartAotRuntime).existsSync()) {
     dartAotRuntime = p.absolute(
         sdkPath,
         Platform.isWindows
-            ? 'dartaotruntime_product.exe'
-            : 'dartaotruntime_product');
-    snapshotName = _resolvePath('gen/dartdevc_aot.dart.snapshot');
+            ? 'dart_precompiled_runtime.exe'
+            : 'dart_precompiled_runtime');
   }
   final executableArgs = <String>[
-    snapshotName,
+    _resolvePath('gen/dartdevc_aot.dart.snapshot'),
     '--sound-null-safety',
     '--dart-sdk-summary',
     _resolvePath('ddc_outline.dill'),

pkg/dynamic_modules/test/runner/util.dart

@@ -86,8 +86,9 @@ Uri genSnapshotBin =
     buildRootUri.resolve(useProduct ? 'gen_snapshot_product' : 'gen_snapshot');
 Uri dart2bytecodeSnapshot =
     buildRootUri.resolve('gen/dart2bytecode.dart.snapshot');
-Uri aotRuntimeBin = buildRootUri
-    .resolve(useProduct ? 'dartaotruntime_product' : 'dartaotruntime');
+Uri aotRuntimeBin = buildRootUri.resolve(useProduct
+    ? 'dart_precompiled_runtime_product'
+    : 'dart_precompiled_runtime');
 Uri vmPlatformDill = buildRootUri.resolve('vm_platform_strong.dill');
 
 // Encodes test results in the format expected by Dart's CI infrastructure.

pkg/front_end/lib/src/linux_and_intel_specific_perf.dart

@@ -41,7 +41,7 @@ int _perfPid = -1;
 /// You might also want the dartaotruntime to include debug symbols:
 ///
 /// ```
-/// cp out/ReleaseX64/dartaotruntime_product \
+/// cp out/ReleaseX64/dart_precompiled_runtime_product \
 ///   out/ReleaseX64/dart-sdk/bin/dartaotruntime
 /// ```
 ///

pkg/smith/lib/configuration.dart

@@ -251,7 +251,7 @@ class Configuration {
           .pathSegments
           .lastWhere((e) => e.isNotEmpty);
       final executableNoExtension = executableName.split('.').first;
-      if (executableNoExtension == 'dartaotruntime') {
+      if (executableNoExtension == 'dart_precompiled_runtime') {
         runtime = Runtime.dartPrecompiled;
       } else if (executableNoExtension == 'dart') {
         runtime = Runtime.vm;

pkg/test_runner/lib/src/build_configurations.dart

@@ -80,7 +80,7 @@ List<String> _selectBuildTargets(Configuration inner) {
   final compiler = inner.compiler;
   const targetsForCompilers = {
     Compiler.dartk: ['runtime'],
-    Compiler.dartkp: ['runtime', 'runtime_precompiled'],
+    Compiler.dartkp: ['runtime', 'dart_precompiled_runtime'],
     Compiler.appJitk: ['runtime'],
     Compiler.fasta: ['create_sdk', 'ddc_stable_test', 'kernel_platform_files'],
     Compiler.ddc: ['ddc_stable_test'],