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

Side by Side Diff: webrtc/BUILD.gn

Issue 1181373004: Add a [rtc_]build_with_neon variable to unify conditions. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Rewording. Created 5 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 | webrtc/build/arm_neon.gypi » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2014 The WebRTC project authors. All Rights Reserved. 1 # Copyright (c) 2014 The WebRTC project authors. All Rights Reserved.
2 # 2 #
3 # Use of this source code is governed by a BSD-style license 3 # Use of this source code is governed by a BSD-style license
4 # that can be found in the LICENSE file in the root of the source 4 # that can be found in the LICENSE file in the root of the source
5 # tree. An additional intellectual property rights grant can be found 5 # tree. An additional intellectual property rights grant can be found
6 # in the file PATENTS. All contributing project authors may 6 # in the file PATENTS. All contributing project authors may
7 # be found in the AUTHORS file in the root of the source tree. 7 # be found in the AUTHORS file in the root of the source tree.
8 8
9 # TODO(kjellander): Rebase this to webrtc/build/common.gypi changes after r6330. 9 # TODO(kjellander): Rebase this to webrtc/build/common.gypi changes after r6330.
10 10
(...skipping 146 matching lines...) Expand 10 before | Expand all | Expand 10 after
157 # functions. Ensure they are disabled for all compilers. 157 # functions. Ensure they are disabled for all compilers.
158 cflags += [ 158 cflags += [
159 "-fno-builtin-cos", 159 "-fno-builtin-cos",
160 "-fno-builtin-sin", 160 "-fno-builtin-sin",
161 "-fno-builtin-cosf", 161 "-fno-builtin-cosf",
162 "-fno-builtin-sinf", 162 "-fno-builtin-sinf",
163 ] 163 ]
164 } 164 }
165 } 165 }
166 166
167 # Add to ARM NEON targets.
168 config("arm_neon_config") {
Andrew MacDonald 2015/06/17 17:44:16 kjellander: I'm not super familiar with this stuff
169 if (current_cpu == "arm") {
170 # Enable compilation for the ARM v7 Neon instruction set. This is needed
171 # since //build/config/arm.gni only enables Neon for iOS, not Android.
172 # This provides the same functionality as webrtc/build/arm_neon.gypi.
173 configs -= [ "//build/config/compiler:compiler_arm_fpu" ]
174 cflags = [ "-mfpu=neon" ]
175 }
176
177 # Disable LTO due to compiler bug.
178 if (rtc_use_lto) {
179 cflags -= [
180 "-flto",
181 "-ffat-lto-objects",
182 ]
183 }
184 }
185
167 source_set("webrtc") { 186 source_set("webrtc") {
168 sources = [ 187 sources = [
169 "call.h", 188 "call.h",
170 "config.h", 189 "config.h",
171 "experiments.h", 190 "experiments.h",
172 "frame_callback.h", 191 "frame_callback.h",
173 "transport.h", 192 "transport.h",
174 ] 193 ]
175 194
176 configs += [ ":common_config" ] 195 configs += [ ":common_config" ]
(...skipping 54 matching lines...) Expand 10 before | Expand all | Expand 10 after
231 250
232 configs += [ ":common_config" ] 251 configs += [ ":common_config" ]
233 public_configs = [ ":common_inherited_config" ] 252 public_configs = [ ":common_inherited_config" ]
234 } 253 }
235 254
236 source_set("gtest_prod") { 255 source_set("gtest_prod") {
237 sources = [ 256 sources = [
238 "test/testsupport/gtest_prod_util.h", 257 "test/testsupport/gtest_prod_util.h",
239 ] 258 ]
240 } 259 }
OLDNEW
« no previous file with comments | « no previous file | webrtc/build/arm_neon.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698