Skip to content

V
V8
Commit e6f62a41 authored by Michael Achenbach's avatar Michael Achenbach Committed by Commit Bot
Browse files
Options

[build] Remove jumbo build configs 

Bug: chromium:994387
Change-Id: I7b37a416ce6cc79903fe04923e86af1d0065ac37
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1890090Reviewed-by: 's avatarMaya Lekova <mslekova@chromium.org>
Reviewed-by: 's avatarYang Guo <yangguo@chromium.org>
Commit-Queue: Michael Achenbach <machenbach@chromium.org>
Cr-Commit-Position: refs/heads/master@{#64669}
parent 6032db48
Hide whitespace changes
Inline Side-by-side
Showing with 4 additions and 86 deletions
BUILD.gn
View file @ e6f62a41
......@@ -6,7 +6,6 @@ import("//build/config/android/config.gni")
import("//build/config/arm.gni")
import("//build/config/dcheck_always_on.gni")
import("//build/config/host_byteorder.gni")
import("//build/config/jumbo.gni")
import("//build/config/mips.gni")
import("//build/config/sanitizers/sanitizers.gni")
import("//build_overrides/build.gni")
......@@ -1403,17 +1402,12 @@ if (v8_use_external_startup_data) {
if (v8_use_multi_snapshots) {
deps += [ ":asm_to_inline_asm_trusted" ]
sources += [ "$target_gen_dir/embedded_trusted.cc" ]
jumbo_excluded_sources = [ "$target_gen_dir/embedded_trusted.cc" ]
}
} else {
sources += [ "$target_gen_dir/embedded.S" ]
if (v8_use_multi_snapshots) {
sources += [ "$target_gen_dir/embedded_trusted.S" ]
jumbo_excluded_sources = [
# Duplicated symbols with embedded.S
"$target_gen_dir/embedded_trusted.S",
]
}
}
......@@ -1504,17 +1498,6 @@ v8_source_set("v8_initializers") {
"src/interpreter/interpreter-intrinsics-generator.h",
]
jumbo_excluded_sources = [
# TODO(mostynb@vewd.com): don't exclude these http://crbug.com/752428
"src/builtins/builtins-async-iterator-gen.cc",
"src/builtins/builtins-async-generator-gen.cc",
# These source files take an unusually large amount of time to
# compile. Build them separately to avoid bottlenecks.
"src/builtins/builtins-regexp-gen.cc",
"src/codegen/code-stub-assembler.cc",
]
if (v8_current_cpu == "x86") {
sources += [
### gcmole(arch:ia32) ###
......@@ -2938,19 +2921,6 @@ v8_source_set("v8_base_without_compiler") {
sources += check_header_includes_sources
}
jumbo_excluded_sources = [
# TODO(mostynb@vewd.com): don't exclude these http://crbug.com/752428
"src/profiler/heap-snapshot-generator.cc", # Macro clash in mman-linux.h
# These source files take an unusually large amount of time to
# compile. Build them separately to avoid bottlenecks.
"src/api/api.cc",
"src/heap/heap.cc",
"src/objects/elements.cc",
"src/objects/objects.cc",
"src/parsing/parser.cc",
]
if (v8_current_cpu == "x86") {
sources += [ ### gcmole(arch:ia32) ###
"src/codegen/ia32/assembler-ia32-inl.h",
......@@ -3104,11 +3074,6 @@ v8_source_set("v8_base_without_compiler") {
"src/diagnostics/unwinding-info-win64.h",
]
}
jumbo_excluded_sources += [
# TODO(mostynb@vewd.com): fix this code so it doesn't need
# to be excluded, see the comments inside.
"src/codegen/arm64/instructions-arm64-constants.cc",
]
} else if (v8_current_cpu == "mips" || v8_current_cpu == "mipsel") {
sources += [ ### gcmole(arch:mipsel) ###
"src/codegen/mips/assembler-mips-inl.h",
......
gni/v8.gni
View file @ e6f62a41
......@@ -2,7 +2,6 @@
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
import("//build/config/jumbo.gni")
import("//build/config/sanitizers/sanitizers.gni")
import("//build/config/v8_target_cpu.gni")
import("//build/split_static_library.gni")
......@@ -71,7 +70,7 @@ if (v8_use_multi_snapshots) {
# build configuration. This allows us to set v8_use_multi_snapshots=true on
# all bots, and e.g. no-snapshot bots will automatically do the right thing.
v8_use_multi_snapshots =
v8_use_external_startup_data && !build_with_chromium && !use_jumbo_build
v8_use_external_startup_data && !build_with_chromium
}
if (v8_enable_backtrace == "") {
......@@ -140,9 +139,9 @@ template("v8_source_set") {
defined(v8_static_library) && v8_static_library && is_win) {
link_target_type = "split_static_library"
} else if (defined(v8_static_library) && v8_static_library) {
link_target_type = "jumbo_static_library"
link_target_type = "static_library"
} else {
link_target_type = "jumbo_source_set"
link_target_type = "source_set"
}
target(link_target_type, target_name) {
forward_variables_from(invoker,
......@@ -161,7 +160,7 @@ template("v8_source_set") {
}
template("v8_header_set") {
jumbo_source_set(target_name) {
source_set(target_name) {
forward_variables_from(invoker, "*", [ "configs" ])
configs -= v8_remove_configs
configs += v8_add_configs
......
infra/mb/mb_config.pyl
View file @ e6f62a41
......@@ -62,11 +62,6 @@
'V8 Linux64 - debug - header includes': 'debug_x64_header_includes',
'V8 Linux64 - shared': 'release_x64_shared_verify_heap',
'V8 Linux64 - verify csa': 'release_x64_verify_csa',
# Jumbo.
'V8 Linux64 Jumbo': 'release_x64_jumbo',
'V8 Linux64 Jumbo - debug': 'debug_x64_jumbo',
'V8 Linux64 Jumbo - limited': 'release_x64_jumbo_limited',
'V8 Linux64 Jumbo - limited - debug': 'debug_x64_jumbo_limited',
# Windows.
'V8 Win32 - builder': 'release_x86_minimal_symbols',
'V8 Win32 - debug builder': 'debug_x86_minimal_symbols',
......@@ -226,8 +221,6 @@
'v8_linux64_msan_rel': 'release_simulate_arm64_msan_minimal_symbols',
'v8_linux64_sanitizer_coverage_rel':
'release_x64_asan_minimal_symbols_coverage',
'v8_linux64_jumbo_compile_rel': 'release_x64_jumbo_trybot',
'v8_linux64_jumbo_limited_compile_rel': 'release_x64_jumbo_limited_trybot',
'v8_linux64_tsan_rel': 'release_x64_tsan_minimal_symbols',
'v8_linux64_tsan_isolates_rel_ng':
'release_x64_tsan_minimal_symbols',
......@@ -453,14 +446,6 @@
'release_bot', 'x64', 'ios_simulator'],
'release_x64_internal': [
'release_bot', 'x64', 'v8_snapshot_internal'],
'release_x64_jumbo': [
'release_bot', 'x64', 'jumbo'],
'release_x64_jumbo_trybot': [
'release_trybot', 'x64', 'jumbo'],
'release_x64_jumbo_limited': [
'release_bot', 'x64', 'jumbo_limited'],
'release_x64_jumbo_limited_trybot': [
'release_trybot', 'x64', 'jumbo_limited'],
'release_x64_minimal_symbols': [
'release_bot', 'x64', 'minimal_symbols'],
'release_x64_pointer_compression': [
......@@ -508,10 +493,6 @@
'debug_bot', 'x64', 'gcc', 'v8_check_header_includes'],
'debug_x64_header_includes': [
'debug_bot', 'x64', 'v8_check_header_includes'],
'debug_x64_jumbo': [
'debug_bot', 'x64', 'jumbo'],
'debug_x64_jumbo_limited': [
'debug_bot', 'x64', 'jumbo_limited'],
'debug_x64_minimal_symbols': [
'debug_bot', 'x64', 'minimal_symbols'],
'debug_x64_perfetto': [
......@@ -658,14 +639,6 @@
'gn_args': 'target_cpu="x64" target_os="ios"',
},
'jumbo': {
'gn_args': 'use_jumbo_build=true',
},
'jumbo_limited': {
'gn_args': 'use_jumbo_build=true jumbo_file_merge_limit=50',
},
'lsan': {
'mixins': ['v8_enable_test_features'],
'gn_args': 'is_lsan=true',
......
test/cctest/BUILD.gn
View file @ e6f62a41
......@@ -297,16 +297,6 @@ v8_source_set("cctest_sources") {
"wasm/wasm-run-utils.h",
]
if (use_jumbo_build) {
jumbo_excluded_sources = [
# TODO(mostynb@opera.com): figure out the jumbo issues with these source
# files, and include them in jumbo compilation units.
"interpreter/bytecode-expectations-printer.cc",
"interpreter/test-bytecode-generator.cc",
"test-api.cc",
]
}
if (v8_current_cpu == "arm") {
sources += [ ### gcmole(arch:arm) ###
"assembler-helper-arm.cc",
......
test/unittests/BUILD.gn
View file @ e6f62a41
......@@ -235,15 +235,6 @@ v8_source_set("unittests_sources") {
"zone/zone-unittest.cc",
]
if (use_jumbo_build) {
jumbo_excluded_sources = [
# TODO(mostynb@opera.com): figure out how to make this build in both
# modes. Template issues on windows if we add a functional_unittest
# namespace as with the other files.
"base/functional-unittest.cc",
]
}
if (v8_current_cpu == "arm") {
sources += [
"assembler/turbo-assembler-arm-unittest.cc",
......
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