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: I96ed52994e0d955300c18026032e68003504666d
> Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/389760
> Reviewed-by: Ryan Macnak <[email protected]>
> Commit-Queue: Siva Annamalai <[email protected]>
> Reviewed-by: Alexander Thomas <[email protected]>

Change-Id: I5dc14973f4ee4e577b2c996839d5e497c97fb440
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/393761
Commit-Queue: Siva Annamalai <[email protected]>
Bot-Commit: Rubber Stamper <[email protected]>
Reviewed-by: Ryan Macnak <[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'],

pkg/test_runner/lib/src/compiler_configuration.dart

@@ -1601,7 +1601,7 @@ class BytecodeCompilerConfiguration extends CompilerConfiguration {
 
   String dartAotRuntime() => _useSdk
       ? '${_configuration.buildDirectory}/dart-sdk/bin/dartaotruntime'
-      : '${_configuration.buildDirectory}/dartaotruntime';
+      : '${_configuration.buildDirectory}/dart_precompiled_runtime';
 
   String dart2bytecodeSnapshot() => _useSdk
       ? '${_configuration.buildDirectory}/dart-sdk/bin/snapshots/dart2bytecode.dart.snapshot'

pkg/test_runner/lib/src/process_queue.dart

@@ -646,17 +646,19 @@ class CommandExecutorImpl implements CommandExecutor {
     steps.add(() => device.runAdbShellCommand(['rm', '-Rf', deviceTestDir]));
     steps.add(() => device.runAdbShellCommand(['mkdir', '-p', deviceTestDir]));
     steps.add(() => device.pushCachedData(
-        '$buildPath/exe.stripped/dartaotruntime', '$devicedir/dartaotruntime'));
+        '$buildPath/exe.stripped/dart_precompiled_runtime',
+        '$devicedir/dart_precompiled_runtime'));
     steps.add(() => device.pushCachedData(
-        '$buildPath/dartaotruntime.sym', '$devicedir/dartaotruntime.sym'));
+        '$buildPath/dart_precompiled_runtime.sym',
+        '$devicedir/dart_precompiled_runtime.sym'));
     steps.add(
         () => device.pushCachedData(processTest, '$devicedir/process_test'));
     steps.add(() => device.pushCachedData(
         abstractSocketTest, '$devicedir/abstract_socket_test'));
     steps.add(() => device.runAdbShellCommand([
           'chmod',
           '777',
-          '$devicedir/dartaotruntime $devicedir/process_test $devicedir/abstract_socket_test'
+          '$devicedir/dart_precompiled_runtime $devicedir/process_test $devicedir/abstract_socket_test'
         ]));
 
     steps.addAll(_pushLibraries(command, device, devicedir, deviceTestDir));
@@ -669,7 +671,7 @@ class CommandExecutorImpl implements CommandExecutor {
     steps.add(() => device.runAdbShellCommand([
           'export LD_LIBRARY_PATH=\$LD_LIBRARY_PATH:$deviceTestDir;'
               'export TEST_COMPILATION_DIR=$deviceTestDir;'
-              '$devicedir/dartaotruntime',
+              '$devicedir/dart_precompiled_runtime',
           '--android-log-to-stderr',
           ...arguments,
         ], timeout: timeoutDuration));

pkg/test_runner/lib/src/runtime_configuration.dart

@@ -118,7 +118,7 @@ abstract class RuntimeConfiguration {
       // cannot.
       dir = dir.replaceAll("SIMARM_X64", "SIMARM");
 
-      dartExecutable = '$dir/dartaotruntime$executableExtension';
+      dartExecutable = '$dir/dart_precompiled_runtime$executableExtension';
     }
 
     TestUtils.ensureExists(dartExecutable, _configuration);
@@ -431,7 +431,7 @@ class DartPrecompiledRuntimeConfiguration extends DartVmRuntimeConfiguration {
     if (script != null &&
         type != 'application/dart-precompiled' &&
         type != 'application/dart-bytecode') {
-      throw "dartaotruntime cannot run files of type '$type'.";
+      throw "dart_precompiled cannot run files of type '$type'.";
     }
 
     var executable = dartPrecompiledBinaryFileName;
@@ -496,7 +496,7 @@ class DartPrecompiledAdbRuntimeConfiguration
     var script = artifact?.filename;
     var type = artifact?.mimeType;
     if (script != null && type != 'application/dart-precompiled') {
-      throw "dartaotruntime cannot run files of type '$type'.";
+      throw "dart_precompiled cannot run files of type '$type'.";
     }
 
     var processTest = processTestBinaryFileName;

pkg/vm/tool/dart_precompiled_runtime2

@@ -33,4 +33,4 @@ fi
 export DART_CONFIGURATION=${DART_CONFIGURATION:-ReleaseX64}
 BIN_DIR="$OUT_DIR$DART_CONFIGURATION"
 
-exec "$BIN_DIR"/dartaotruntime "$@"
+exec "$BIN_DIR"/dart_precompiled_runtime "$@"

runtime/BUILD.gn

@@ -51,7 +51,7 @@ config("dart_product_config") {
   }
 }
 
-config("dart_aotruntime_config") {
+config("dart_precompiled_runtime_config") {
   defines = []
   defines += [
     "DART_PRECOMPILED_RUNTIME",

runtime/bin/BUILD.gn

@@ -903,21 +903,15 @@ dart_executable("dart") {
   }
 }
 
-# This rule is here to ensure benchmark suites do not break, it can be
-# removed once the benchmark suites are updated.
-group("dart_precompiled_runtime") {
-  deps = [ ":dartaotruntime" ]
-}
-
-dart_executable("dartaotruntime") {
+dart_executable("dart_precompiled_runtime") {
   extra_configs = [
-    "..:dart_aotruntime_config",
+    "..:dart_precompiled_runtime_config",
     "..:add_empty_macho_section_config",
   ]
   extra_deps = [
     ":icudtl_cc",
-    "..:libdart_aotruntime",
-    "../platform:libdart_platform_aotruntime",
+    "..:libdart_precompiled_runtime",
+    "../platform:libdart_platform_precompiled_runtime",
   ]
   if (dart_runtime_mode != "release") {
     extra_deps += [ "../observatory:standalone_observatory_archive" ]
@@ -950,15 +944,15 @@ dart_executable("dartaotruntime") {
   }
 }
 
-dart_executable("dartaotruntime_product") {
+dart_executable("dart_precompiled_runtime_product") {
   use_product_mode = true
   extra_configs = [
-    "..:dart_aotruntime_config",
+    "..:dart_precompiled_runtime_config",
     "..:add_empty_macho_section_config",
   ]
   extra_deps = [
-    "..:libdart_aotruntime_product",
-    "../platform:libdart_platform_aotruntime_product",
+    "..:libdart_precompiled_runtime_product",
+    "../platform:libdart_platform_precompiled_runtime_product",
   ]
   extra_sources = [
     "builtin.cc",
@@ -984,17 +978,17 @@ dart_executable("dartaotruntime_product") {
 if (build_analyze_snapshot) {
   dart_executable("analyze_snapshot") {
     use_product_mode = dart_runtime_mode == "release"
-    extra_configs = [ "..:dart_aotruntime_config" ]
+    extra_configs = [ "..:dart_precompiled_runtime_config" ]
 
     if (use_product_mode) {
       extra_deps = [
-        "..:libdart_aotruntime_product",
-        "../platform:libdart_platform_aotruntime_product",
+        "..:libdart_precompiled_runtime_product",
+        "../platform:libdart_platform_precompiled_runtime_product",
       ]
     } else {
       extra_deps = [
-        "..:libdart_aotruntime",
-        "../platform:libdart_platform_aotruntime",
+        "..:libdart_precompiled_runtime",
+        "../platform:libdart_platform_precompiled_runtime",
       ]
     }
 

runtime/bin/dartaotruntime_test_component.cml renamed to runtime/bin/dart_precompiled_runtime_test_component.cml

@@ -7,7 +7,7 @@
     "//runtime/vm.shard.cml",
   ],
   program: {
-    binary: "exe.stripped/dartaotruntime",
+    binary: "exe.stripped/dart_precompiled_runtime",
     runner: "elf_test_runner",
   },
   capabilities: [

runtime/bin/entrypoints_verification_test.cc

@@ -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"
 #include "include/dart_native_api.h"
@@ -29,12 +29,12 @@
     abort();                                                                   \
   }
 
-static bool is_dartaotruntime = true;
+static bool is_dart_precompiled_runtime = true;
 
 // Some invalid accesses are allowed in AOT since we don't retain @pragma
 // annotations. Therefore we skip the negative tests in AOT.
 #define FAIL(name, result)                                                     \
-  if (!is_dartaotruntime) {                                                    \
+  if (!is_dart_precompiled_runtime) {                                          \
     Fail(name, result);                                                        \
   }
 
@@ -46,7 +46,7 @@ void Fail(const char* name, Dart_Handle result) {
 }
 
 #define FAIL_INVOKE_FIELD(name, result)                                        \
-  if (!is_dartaotruntime) {                                                    \
+  if (!is_dart_precompiled_runtime) {                                          \
     FailInvokeField(name, result);                                             \
   }
 
@@ -94,7 +94,7 @@ static void TestFields(Dart_Handle target) {
 }
 
 DART_EXPORT void RunTests() {
-  is_dartaotruntime = Dart_IsPrecompiledRuntime();
+  is_dart_precompiled_runtime = Dart_IsPrecompiledRuntime();
 
   Dart_Handle lib = Dart_RootLibrary();
 

runtime/bin/ffi_test/ffi_test_functions_vmspecific.cc

@@ -34,7 +34,7 @@
 #include <limits>
 
 // 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"
 #include "include/dart_native_api.h"