Commit 6a55a37d authored by Simon Zünd's avatar Simon Zünd Committed by Commit Bot

[Cleanup] Integrate v8_nosnapshot into mksnapshot

This CL integrates the "v8_nosnapshot" source set into the "mksnapshot"
binary. The "mksnapshot" binary is the last use-site after unconditionally
enabling snapshot and embedded builtins.

Bug: v8:8519
Change-Id: I8e737e10a3f7729e18ac71733b811bf399ebf330
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1873685Reviewed-by: 's avatarMichael Achenbach <machenbach@chromium.org>
Commit-Queue: Simon Zünd <szuend@chromium.org>
Cr-Commit-Position: refs/heads/master@{#64494}
parent 2205ab3a
...@@ -1338,25 +1338,6 @@ source_set("v8_maybe_snapshot") { ...@@ -1338,25 +1338,6 @@ source_set("v8_maybe_snapshot") {
} }
} }
v8_source_set("v8_nosnapshot") {
visibility = [ ":*" ] # Only targets in this file can depend on this.
deps = [
":torque_generated_initializers",
]
public_deps = [
":v8_maybe_icu",
]
sources = [
"src/snapshot/embedded/embedded-empty.cc",
"src/snapshot/snapshot-empty.cc",
]
configs = [ ":internal_config" ]
}
if (!v8_use_external_startup_data) { if (!v8_use_external_startup_data) {
v8_source_set("v8_snapshot") { v8_source_set("v8_snapshot") {
# Only targets in this file and the top-level visibility target can # Only targets in this file and the top-level visibility target can
...@@ -3811,6 +3792,7 @@ if (current_toolchain == v8_snapshot_toolchain) { ...@@ -3811,6 +3792,7 @@ if (current_toolchain == v8_snapshot_toolchain) {
visibility = [ ":*" ] # Only targets in this file can depend on this. visibility = [ ":*" ] # Only targets in this file can depend on this.
sources = [ sources = [
"src/snapshot/embedded/embedded-empty.cc",
"src/snapshot/embedded/embedded-file-writer.cc", "src/snapshot/embedded/embedded-file-writer.cc",
"src/snapshot/embedded/embedded-file-writer.h", "src/snapshot/embedded/embedded-file-writer.h",
"src/snapshot/embedded/platform-embedded-file-writer-aix.cc", "src/snapshot/embedded/platform-embedded-file-writer-aix.cc",
...@@ -3824,6 +3806,7 @@ if (current_toolchain == v8_snapshot_toolchain) { ...@@ -3824,6 +3806,7 @@ if (current_toolchain == v8_snapshot_toolchain) {
"src/snapshot/embedded/platform-embedded-file-writer-win.cc", "src/snapshot/embedded/platform-embedded-file-writer-win.cc",
"src/snapshot/embedded/platform-embedded-file-writer-win.h", "src/snapshot/embedded/platform-embedded-file-writer-win.h",
"src/snapshot/mksnapshot.cc", "src/snapshot/mksnapshot.cc",
"src/snapshot/snapshot-empty.cc",
] ]
configs = [ ":internal_config" ] configs = [ ":internal_config" ]
...@@ -3834,7 +3817,7 @@ if (current_toolchain == v8_snapshot_toolchain) { ...@@ -3834,7 +3817,7 @@ if (current_toolchain == v8_snapshot_toolchain) {
":v8_init", ":v8_init",
":v8_libbase", ":v8_libbase",
":v8_libplatform", ":v8_libplatform",
":v8_nosnapshot", ":v8_maybe_icu",
"//build/win:default_exe_manifest", "//build/win:default_exe_manifest",
] ]
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment