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

Side by Side Diff: build/config/BUILDCONFIG.gn

Issue 431803003: gn win: Enable exceptions for pdfium build (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: include pdfium roll Created 6 years, 4 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 | Annotate | Revision Log
« no previous file with comments | « DEPS ('k') | build/config/compiler/BUILD.gn » ('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) 2013 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2013 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 # ============================================================================= 5 # =============================================================================
6 # BUILD FLAGS 6 # BUILD FLAGS
7 # ============================================================================= 7 # =============================================================================
8 # 8 #
9 # This block lists input arguments to the build, along with their default 9 # This block lists input arguments to the build, along with their default
10 # values. GN requires listing them explicitly so it can validate input and have 10 # values. GN requires listing them explicitly so it can validate input and have
(...skipping 306 matching lines...) Expand 10 before | Expand all | Expand 10 after
317 "//build/config/compiler:no_rtti", 317 "//build/config/compiler:no_rtti",
318 "//build/config/compiler:runtime_library", 318 "//build/config/compiler:runtime_library",
319 ] 319 ]
320 if (is_win) { 320 if (is_win) {
321 _native_compiler_configs += [ 321 _native_compiler_configs += [
322 "//build/config/win:lean_and_mean", 322 "//build/config/win:lean_and_mean",
323 "//build/config/win:nominmax", 323 "//build/config/win:nominmax",
324 "//build/config/win:sdk", 324 "//build/config/win:sdk",
325 "//build/config/win:unicode", 325 "//build/config/win:unicode",
326 ] 326 ]
327 if (is_component_build) {
328 _native_compiler_configs += [ "//build/config/compiler:exceptions" ]
Nico 2014/07/31 23:15:23 This isn't right, see https://codereview.chromium.
scottmg 2014/07/31 23:24:31 OK, this is mostly just a port of existing behavio
329 } else {
330 # We don't use exceptions, and when we link statically we can just get
331 # rid of them entirely.
332 _native_compiler_configs += [ "//build/config/compiler:no_exceptions" ]
333 }
327 } 334 }
328 if (is_posix) { 335 if (is_posix) {
329 _native_compiler_configs += [ 336 _native_compiler_configs += [
330 "//build/config/gcc:no_exceptions", 337 "//build/config/compiler:no_exceptions",
331 "//build/config/gcc:symbol_visibility_hidden", 338 "//build/config/gcc:symbol_visibility_hidden",
332 ] 339 ]
333 } 340 }
334 341
335 if (is_linux) { 342 if (is_linux) {
336 _native_compiler_configs += [ "//build/config/linux:sdk", ] 343 _native_compiler_configs += [ "//build/config/linux:sdk", ]
337 } else if (is_mac) { 344 } else if (is_mac) {
338 _native_compiler_configs += [ "//build/config/mac:sdk", ] 345 _native_compiler_configs += [ "//build/config/mac:sdk", ]
339 } else if (is_ios) { 346 } else if (is_ios) {
340 _native_compiler_configs += [ "//build/config/ios:sdk", ] 347 _native_compiler_configs += [ "//build/config/ios:sdk", ]
(...skipping 129 matching lines...) Expand 10 before | Expand all | Expand 10 after
470 host_toolchain = "//build/toolchain/linux:$build_cpu_arch" 477 host_toolchain = "//build/toolchain/linux:$build_cpu_arch"
471 set_default_toolchain("//build/toolchain/linux:$cpu_arch") 478 set_default_toolchain("//build/toolchain/linux:$cpu_arch")
472 } 479 }
473 } else if (is_mac) { 480 } else if (is_mac) {
474 host_toolchain = "//build/toolchain/mac:clang" 481 host_toolchain = "//build/toolchain/mac:clang"
475 set_default_toolchain(host_toolchain) 482 set_default_toolchain(host_toolchain)
476 } else if (is_ios) { 483 } else if (is_ios) {
477 host_toolchain = "//build/toolchain/mac:host_clang" 484 host_toolchain = "//build/toolchain/mac:host_clang"
478 set_default_toolchain("//build/toolchain/mac:clang") 485 set_default_toolchain("//build/toolchain/mac:clang")
479 } 486 }
OLDNEW
« no previous file with comments | « DEPS ('k') | build/config/compiler/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698