Commit 38b31043 authored by machenbach's avatar machenbach Committed by Commit bot

[gn] Switch linux64 debug and internal snapshot to gn

BUG=chromium:474921

Committed: https://crrev.com/3cfc9f209ed0c173319659114859b7e848812fd7
Review-Url: https://codereview.chromium.org/2105353002
Cr-Original-Commit-Position: refs/heads/master@{#37414}
Cr-Commit-Position: refs/heads/master@{#37425}
parent cede9ce5
......@@ -101,6 +101,11 @@ template("v8_isolate_run") {
} else {
use_snapshot = "false"
}
if (v8_has_valgrind) {
has_valgrind = "1"
} else {
has_valgrind = "0"
}
# Note, all paths will be rebased in isolate_driver.py to be relative to
# the isolate file.
......@@ -132,7 +137,7 @@ template("v8_isolate_run") {
"--config-variable",
"gcmole=0",
"--config-variable",
"has_valgrind=0",
"has_valgrind=$has_valgrind",
"--config-variable",
"icu_use_data_file_flag=$icu_use_data_file_flag",
"--config-variable",
......
......@@ -6,6 +6,10 @@ import("//build/config/v8_target_cpu.gni")
import("//build/config/sanitizers/sanitizers.gni")
declare_args() {
# Indicate if valgrind was fetched as a custom deps to make it available on
# swarming.
v8_has_valgrind = false
# Turns on compiler optimizations in V8 in Debug build.
v8_optimized_debug = true
......
......@@ -26,9 +26,9 @@
'V8 Linux - noi18n - debug': 'gyp_debug_x86_no_i18n',
# Linux64.
'V8 Linux64 - builder': 'gn_release_x64',
'V8 Linux64 - debug builder': 'gyp_debug_x64_valgrind',
'V8 Linux64 - debug builder': 'gn_debug_x64_valgrind',
'V8 Linux64 - custom snapshot - debug builder': 'gyp_debug_x64_custom',
'V8 Linux64 - internal snapshot': 'gyp_release_x64_internal',
'V8 Linux64 - internal snapshot': 'gn_release_x64_internal',
# Windows.
'V8 Win32 - builder': 'gyp_release_x86_minimal_symbols',
'V8 Win32 - debug builder': 'gyp_debug_x86_minimal_symbols',
......@@ -141,7 +141,7 @@
'v8_linux_gcc_rel': 'gyp_release_x86_gcc_minimal_symbols',
'v8_linux64_rel_ng': 'gn_release_x64_trybot',
'v8_linux64_avx2_rel_ng': 'gn_release_x64_trybot',
'v8_linux64_avx2_dbg': 'gyp_debug_x64_trybot',
'v8_linux64_avx2_dbg': 'gn_debug_x64_trybot',
'v8_linux64_asan_rel_ng': 'gyp_release_x64_asan_minimal_symbols',
'v8_linux64_msan_rel': 'gyp_release_simulate_arm64_msan_minimal_symbols',
'v8_linux64_sanitizer_coverage_rel':
......@@ -180,9 +180,17 @@
# GN release configs for x64.
'gn_release_x64': [
'gn', 'release_bot', 'x64', 'swarming'],
'gn_release_x64_internal': [
'gn', 'release_bot', 'x64', 'swarming', 'v8_snapshot_internal'],
'gn_release_x64_trybot': [
'gn', 'release_trybot', 'x64', 'swarming'],
# GN debug configs for x64.
'gn_debug_x64_trybot': [
'gn', 'debug_trybot', 'x64', 'swarming'],
'gn_debug_x64_valgrind': [
'gn', 'debug_bot', 'x64', 'swarming', 'valgrind'],
# Gyp debug configs for arm.
'gyp_debug_arm': [
'gyp', 'debug_bot', 'arm', 'crosscompile', 'hard_float', 'swarming'],
......@@ -212,10 +220,6 @@
'gyp', 'debug_bot', 'x64', 'swarming', 'v8_snapshot_custom'],
'gyp_debug_x64_minimal_symbols': [
'gyp', 'debug_bot', 'x64', 'minimal_symbols', 'swarming'],
'gyp_debug_x64_trybot': [
'gyp', 'debug_trybot', 'x64', 'swarming'],
'gyp_debug_x64_valgrind': [
'gyp', 'debug_bot', 'x64', 'swarming', 'valgrind'],
# Gyp debug configs for x86.
'gyp_debug_x86': [
......@@ -302,8 +306,6 @@
'gyp', 'release_bot', 'x64', 'tsan', 'swarming'],
'gyp_release_x64_tsan_minimal_symbols': [
'gyp', 'release_bot', 'x64', 'tsan', 'minimal_symbols', 'swarming'],
'gyp_release_x64_internal': [
'gyp', 'release_bot', 'x64', 'swarming', 'v8_snapshot_internal'],
'gyp_release_x64_trybot': [
'gyp', 'release_trybot', 'x64', 'swarming'],
......@@ -542,8 +544,7 @@
},
'valgrind': {
# TODO(machenbach): Add this to gn.
'gn_args': 'has_valgrind=true',
'gn_args': 'v8_has_valgrind=true',
'gyp_defines': 'has_valgrind=1',
},
......
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