Commit 33ceb9e5 authored by machenbach's avatar machenbach Committed by Commit bot

Revert of [gn] Switch custom snapsot bot to gn (patchset #2 id:20001 of...

Revert of [gn] Switch custom snapsot bot to gn (patchset #2 id:20001 of https://codereview.chromium.org/2113583002/ )

Reason for revert:
Breaks...

Original issue's description:
> [gn] Switch custom snapsot bot to gn
>
> BUG=chromium:474921
> NOTRY=true
>
> Committed: https://crrev.com/52a43518abe9143a14ded92a89bd3384a47e9193
> Cr-Commit-Position: refs/heads/master@{#37431}

TBR=yangguo@chromium.org,vogelheim@chromium.org,jochen@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chromium:474921

Review-Url: https://codereview.chromium.org/2107423003
Cr-Commit-Position: refs/heads/master@{#37434}
parent 203391bc
......@@ -28,9 +28,6 @@ declare_args() {
# Enable compiler warnings when using V8_DEPRECATE_SOON apis.
v8_imminent_deprecation_warnings = false
# Embeds the given script into the snapshot.
v8_embed_script = ""
# Sets -dENABLE_DISASSEMBLER.
v8_enable_disassembler = false
......@@ -591,8 +588,6 @@ action("run_mksnapshot") {
script = "tools/run.py"
sources = []
outputs = [
"$target_gen_dir/snapshot.cc",
]
......@@ -619,11 +614,6 @@ action("run_mksnapshot") {
rebase_path("$root_out_dir/snapshot_blob.bin", root_build_dir),
]
}
if (v8_embed_script != "") {
sources += [ v8_embed_script ]
args += [ rebase_path(v8_embed_script, root_build_dir) ]
}
}
###############################################################################
......
......@@ -27,7 +27,7 @@
# Linux64.
'V8 Linux64 - builder': 'gn_release_x64',
'V8 Linux64 - debug builder': 'gn_debug_x64_valgrind',
'V8 Linux64 - custom snapshot - debug builder': 'gn_debug_x64_custom',
'V8 Linux64 - custom snapshot - debug builder': 'gyp_debug_x64_custom',
'V8 Linux64 - internal snapshot': 'gn_release_x64_internal',
# Windows.
'V8 Win32 - builder': 'gyp_release_x86_minimal_symbols',
......@@ -186,8 +186,6 @@
'gn', 'release_trybot', 'x64', 'swarming'],
# GN debug configs for x64.
'gn_debug_x64_custom': [
'gn', 'debug_bot', 'x64', 'swarming', 'v8_snapshot_custom'],
'gn_debug_x64_trybot': [
'gn', 'debug_trybot', 'x64', 'swarming'],
'gn_debug_x64_valgrind': [
......@@ -218,6 +216,8 @@
'gyp', 'debug_bot', 'x64', 'swarming'],
'gyp_debug_x64_asan_edge': [
'gyp', 'debug_bot', 'x64', 'asan', 'lsan', 'edge'],
'gyp_debug_x64_custom': [
'gyp', 'debug_bot', 'x64', 'swarming', 'v8_snapshot_custom'],
'gyp_debug_x64_minimal_symbols': [
'gyp', 'debug_bot', 'x64', 'minimal_symbols', 'swarming'],
......@@ -589,10 +589,8 @@
},
'v8_snapshot_custom': {
# GN path is relative to project root.
'gn_args': 'v8_embed_script=test/mjsunit/mjsunit.js',
# Gyp path is relative to src/v8.gyp.
# TODO(machenbach): Add this to gn.
# Path needs to be relative to src/v8.gyp.
'gyp_defines': 'embed_script=../test/mjsunit/mjsunit.js',
},
......
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