Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(876)

Side by Side Diff: snapshot_toolchain.gni

Issue 2116913002: Land v8-side changes to switch to v8_current_cpu in the GN build. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: remove incorrect assert Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 # Copyright 2015 the V8 project authors. All rights reserved. 1 # Copyright 2015 the V8 project authors. All rights reserved.
2 # Redistribution and use in source and binary forms, with or without 2 # Redistribution and use in source and binary forms, with or without
3 # modification, are permitted provided that the following conditions are 3 # modification, are permitted provided that the following conditions are
4 # met: 4 # met:
5 # 5 #
6 # * Redistributions of source code must retain the above copyright 6 # * Redistributions of source code must retain the above copyright
7 # notice, this list of conditions and the following disclaimer. 7 # notice, this list of conditions and the following disclaimer.
8 # * Redistributions in binary form must reproduce the above 8 # * Redistributions in binary form must reproduce the above
9 # copyright notice, this list of conditions and the following 9 # copyright notice, this list of conditions and the following
10 # disclaimer in the documentation and/or other materials provided 10 # disclaimer in the documentation and/or other materials provided
11 # with the distribution. 11 # with the distribution.
12 # * Neither the name of Google Inc. nor the names of its 12 # * Neither the name of Google Inc. nor the names of its
13 # contributors may be used to endorse or promote products derived 13 # contributors may be used to endorse or promote products derived
14 # from this software without specific prior written permission. 14 # from this software without specific prior written permission.
15 # 15 #
16 # THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS 16 # THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
17 # "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT 17 # "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
18 # LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR 18 # LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
19 # A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT 19 # A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
20 # OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, 20 # OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
21 # SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT 21 # SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
22 # LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, 22 # LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
23 # DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY 23 # DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
24 # THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT 24 # THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
25 # (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE 25 # (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
26 # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 26 # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
27 27
28 import("//build/config/v8_target_cpu.gni")
29
28 declare_args() { 30 declare_args() {
29 # The snapshot needs to be compiled for the host, but compiled with 31 # The snapshot needs to be compiled for the host, but compiled with
30 # a toolchain that matches the bit-width of the target. 32 # a toolchain that matches the bit-width of the target.
31 v8_snapshot_toolchain = "" 33 v8_snapshot_toolchain = ""
32 } 34 }
33 35
34 # TODO(GYP): For now we only support 32-bit little-endian target builds from an 36 # TODO(GYP): For now we only support 32-bit little-endian target builds from an
35 # x64 Linux host. Eventually we need to support all of the host/target 37 # x64 Linux host. Eventually we need to support all of the host/target
36 # configurations v8 runs on. 38 # configurations v8 runs on.
37 if (v8_snapshot_toolchain == "") { 39 if (v8_snapshot_toolchain == "") {
38 if (host_cpu == "x64" && host_os == "linux") { 40 if (host_cpu == "x64" && host_os == "linux") {
39 if (target_cpu == "arm" || target_cpu == "mipsel" || target_cpu == "x86") { 41 if (current_cpu == "arm" || current_cpu == "mipsel" ||
40 if (target_os == "android" || is_clang) { 42 current_cpu == "x86") {
41 v8_snapshot_toolchain = "//build/toolchain/linux:clang_x86" 43 _snapshot_cpu = "x86"
42 } else if (target_os == "chromeos") { 44 } else {
43 # TODO(dpranke): crbug.com/608596: Remove this clause once the 45 assert(current_cpu == "arm64" || current_cpu == "x64" ||
44 # CrOS ebuilds are setting v8_snapshot_toolchain directly and 46 current_cpu == "mipsel64",
lmilko 2016/07/20 12:00:00 current_cpu should be mips64el not mipsel64 Chrom
45 # we've cleaned up the sysroot settings in //build. 47 "Need environment for this arch: $current_cpu")
46 v8_snapshot_toolchain = "//build/toolchain/linux:clang_x86" 48 _snapshot_cpu = "x64"
47 } else { 49 }
48 v8_snapshot_toolchain = "//build/toolchain/linux:x86"
49 }
50 50
51 } else if (target_cpu == "x64" || target_cpu == "arm64" || 51 if (v8_current_cpu != _snapshot_cpu) {
52 target_cpu == "mips64el") { 52 _cpus = "${_snapshot_cpu}_v8_${v8_current_cpu}"
53 if (target_os == "android" || is_clang) {
54 v8_snapshot_toolchain = "//build/toolchain/linux:clang_x64"
55 } else if (target_os == "chromeos") {
56 # TODO(dpranke): crbug.com/608596: Remove this clause once the
57 # CrOS ebuilds are setting v8_snapshot_toolchain directly and
58 # we've cleaned up the sysroot settings in //build.
59 v8_snapshot_toolchain = "//build/toolchain/linux:clang_x64"
60 } else {
61 v8_snapshot_toolchain = "//build/toolchain/linux:x64"
62 }
63 } else { 53 } else {
64 assert(false, "Need environment for this arch: $target_cpu") 54 _cpus = _snapshot_cpu
65 } 55 }
66 } else if (host_os == "mac" && target_os == "win") { 56
67 v8_snapshot_toolchain = "//build/toolchain/mac:clang_$target_cpu" 57 if ((host_os == "linux" && current_os == "android") || is_clang) {
58 v8_snapshot_toolchain = "//build/toolchain/linux:clang_${_cpus}"
59 } else {
60 v8_snapshot_toolchain = "//build/toolchain/linux:${_cpus}"
61 }
62 } else if (host_os == "mac" && current_os == "win") {
63 assert(v8_current_cpu == current_cpu,
64 "v8 target must match the regular target on this platform")
65 v8_snapshot_toolchain = "//build/toolchain/mac:clang_$current_cpu"
68 } else { 66 } else {
69 v8_snapshot_toolchain = default_toolchain 67 v8_snapshot_toolchain = default_toolchain
70 } 68 }
71 } 69 }
72
73 # TODO(dpranke): snapshot_toolchain is provided for backwards compatibility
74 # and should be removed once all callers are updated to refer to
75 # v8_snapshot_toolchain directly.
76 snapshot_toolchain = v8_snapshot_toolchain
OLDNEW
« build_overrides/v8.gni ('K') | « gni/v8.gni ('k') | test/cctest/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698