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

Side by Side Diff: build/config/arm.gni

Issue 2070653003: Rework v8_target_arch / target_cpu approach to use v8_target_cpu. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 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
« no previous file with comments | « no previous file | build/config/mips.gni » ('j') | build/config/v8_target_cpu.gni » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import("//build/config/v8_target_cpu.gni")
6
5 # These are primarily relevant in current_cpu == "arm" contexts, where 7 # These are primarily relevant in current_cpu == "arm" contexts, where
6 # ARM code is being compiled. But they can also be relevant in the 8 # ARM code is being compiled. But they can also be relevant in the
7 # host toolchain context when target_cpu == "arm", where a host-side 9 # other contexts when the code will change its behavior based on the
8 # build tool being built will change its behavior depending on the 10 # cpu it wants to generate code for.
9 # details of the target configuration. 11 if (v8_target_cpu == "arm" || current_cpu == "arm") {
Nico 2016/06/16 08:17:28 ubernit: maybe swap the operands to || to match th
Dirk Pranke 2016/06/16 17:35:05 Acknowledged.
10 if (target_cpu == "arm" || current_cpu == "arm") {
11 declare_args() { 12 declare_args() {
12 # Version of the ARM processor when compiling on ARM. Ignored on non-ARM 13 # Version of the ARM processor when compiling on ARM. Ignored on non-ARM
13 # platforms. 14 # platforms.
14 arm_version = 7 15 arm_version = 7
15 16
16 # The ARM floating point mode. This is either the string "hard", "soft", or 17 # The ARM floating point mode. This is either the string "hard", "soft", or
17 # "softfp". An empty string means to use the default one for the 18 # "softfp". An empty string means to use the default one for the
18 # arm_version. 19 # arm_version.
19 arm_float_abi = "" 20 arm_float_abi = ""
20 21
(...skipping 58 matching lines...) Expand 10 before | Expand all | Expand 10 after
79 arm_float_abi = "hard" 80 arm_float_abi = "hard"
80 } 81 }
81 } 82 }
82 83
83 if (arm_use_neon) { 84 if (arm_use_neon) {
84 arm_fpu = "neon" 85 arm_fpu = "neon"
85 } else { 86 } else {
86 arm_fpu = "vfpv3-d16" 87 arm_fpu = "vfpv3-d16"
87 } 88 }
88 } 89 }
89 } else if (current_cpu == "arm64" || target_cpu == "arm64") { 90 } else if (current_cpu == "arm64" || v8_target_cpu == "arm64") {
Michael Achenbach 2016/06/16 07:49:55 ocd nit: maybe turn the conditions around like for
Dirk Pranke 2016/06/16 17:35:05 Ack.
90 # arm64 supports only "hard". 91 # arm64 supports only "hard".
91 arm_float_abi = "hard" 92 arm_float_abi = "hard"
92 arm_use_neon = true 93 arm_use_neon = true
93 } 94 }
OLDNEW
« no previous file with comments | « no previous file | build/config/mips.gni » ('j') | build/config/v8_target_cpu.gni » ('J')

Powered by Google App Engine
This is Rietveld 408576698