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

Side by Side Diff: build/config/nacl/BUILD.gn

Issue 1434823002: Build NaCl browser tests with GN (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Move the nacl group to the top of the file Created 5 years, 1 month 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 | chrome/test/BUILD.gn » ('j') | chrome/test/data/nacl/BUILD.gn » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2014 The Native Client Authors. All rights reserved. 1 # Copyright (c) 2014 The Native Client 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/nacl/config.gni") 5 import("//build/config/nacl/config.gni")
6 6
7 # Native Client Definitions 7 # Native Client Definitions
8 config("nacl_defines") { 8 config("nacl_defines") {
9 if (is_linux || is_android || is_nacl) { 9 if (is_linux || is_android || is_nacl) {
10 defines = [ 10 defines = [
11 "_POSIX_C_SOURCE=199506", 11 "_POSIX_C_SOURCE=199506",
12 "_XOPEN_SOURCE=600", 12 "_XOPEN_SOURCE=600",
13 "_GNU_SOURCE=1", 13 "_GNU_SOURCE=1",
14 "__STDC_LIMIT_MACROS=1", 14 "__STDC_LIMIT_MACROS=1",
15 ] 15 ]
16 } else if (is_win) { 16 } else if (is_win) {
17 defines = [ "__STDC_LIMIT_MACROS=1" ] 17 defines = [ "__STDC_LIMIT_MACROS=1" ]
18 } 18 }
19
20 if (current_cpu == "pnacl" && !is_nacl_nonsfi) {
21 defines += [ "NACL_BUILD_ARCH=pnacl" ]
Roland McGrath 2015/11/18 20:36:58 Looks like this should be NACL_pnacl, not just pna
Petr Hosek 2015/11/18 23:05:33 pnacl is correct in this case, NACL_ARCH macro pre
22 }
19 } 23 }
20 24
21 config("nexe_defines") { 25 config("nexe_defines") {
22 defines = [ 26 defines = [
23 "DYNAMIC_ANNOTATIONS_ENABLED=1", 27 "DYNAMIC_ANNOTATIONS_ENABLED=1",
24 "DYNAMIC_ANNOTATIONS_PREFIX=NACL_", 28 "DYNAMIC_ANNOTATIONS_PREFIX=NACL_",
25 ] 29 ]
26 } 30 }
27 31
28 config("nacl_warnings") { 32 config("nacl_warnings") {
(...skipping 99 matching lines...) Expand 10 before | Expand all | Expand 10 after
128 "-Os", 132 "-Os",
129 133
130 # These are omitted from non-IRT libraries to keep the libraries 134 # These are omitted from non-IRT libraries to keep the libraries
131 # themselves small. 135 # themselves small.
132 "-ffunction-sections", 136 "-ffunction-sections",
133 "-fdata-sections", 137 "-fdata-sections",
134 ] 138 ]
135 139
136 ldflags = [ "-Wl,--gc-sections" ] 140 ldflags = [ "-Wl,--gc-sections" ]
137 } 141 }
OLDNEW
« no previous file with comments | « no previous file | chrome/test/BUILD.gn » ('j') | chrome/test/data/nacl/BUILD.gn » ('J')

Powered by Google App Engine
This is Rietveld 408576698