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

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

Issue 2648543003: Android: Add no-op configs for hiding symbols by default (Closed)
Patch Set: Created 3 years, 11 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 | « build/config/BUILDCONFIG.gn ('k') | no next file » | no next file with comments »
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/android/config.gni") 5 import("//build/config/android/config.gni")
6 import("//build/config/sanitizers/sanitizers.gni") 6 import("//build/config/sanitizers/sanitizers.gni")
7 7
8 assert(is_android) 8 assert(is_android)
9 9
10 # This is included by reference in the //build/config/compiler config that 10 # This is included by reference in the //build/config/compiler config that
(...skipping 182 matching lines...) Expand 10 before | Expand all | Expand 10 after
193 cflags = [ "-fPIE" ] 193 cflags = [ "-fPIE" ]
194 asmflags = [ "-fPIE" ] 194 asmflags = [ "-fPIE" ]
195 ldflags = [ "-pie" ] 195 ldflags = [ "-pie" ]
196 } 196 }
197 197
198 config("hide_native_jni_exports") { 198 config("hide_native_jni_exports") {
199 ldflags = [ "-Wl,--version-script=" + 199 ldflags = [ "-Wl,--version-script=" +
200 rebase_path("//build/android/android_no_jni_exports.lst") ] 200 rebase_path("//build/android/android_no_jni_exports.lst") ]
201 } 201 }
202 202
203 config("hide_all_but_jni_onload") {
204 # TODO(agrieve): https://codereview.chromium.org/2633593004/
205 }
206
207 config("hide_all_but_jni") {
208 # TODO(agrieve): https://codereview.chromium.org/2633593004/
209 }
210
203 # Instrumentation ------------------------------------------------------------- 211 # Instrumentation -------------------------------------------------------------
204 # 212 #
205 # The BUILDCONFIG file sets the "default_cygprofile_instrumentation" config on 213 # The BUILDCONFIG file sets the "default_cygprofile_instrumentation" config on
206 # targets by default. You can override whether the cygprofile instrumentation is 214 # targets by default. You can override whether the cygprofile instrumentation is
207 # used on a per-target basis: 215 # used on a per-target basis:
208 # 216 #
209 # configs -= [ "//build/config/android:default_cygprofile_instrumentation" ] 217 # configs -= [ "//build/config/android:default_cygprofile_instrumentation" ]
210 # configs += [ "//build/config/android:no_cygprofile_instrumentation" ] 218 # configs += [ "//build/config/android:no_cygprofile_instrumentation" ]
211 219
212 config("default_cygprofile_instrumentation") { 220 config("default_cygprofile_instrumentation") {
(...skipping 15 matching lines...) Expand all
228 236
229 # Avoid errors with current NDK: 237 # Avoid errors with current NDK:
230 # "third_party/android_tools/ndk/toolchains/arm-linux-androideabi-4.6/preb uilt/linux-x86_64/bin/../lib/gcc/arm-linux-androideabi/4.6/include/arm_neon.h:34 26:3: error: argument must be a constant" 238 # "third_party/android_tools/ndk/toolchains/arm-linux-androideabi-4.6/preb uilt/linux-x86_64/bin/../lib/gcc/arm-linux-androideabi/4.6/include/arm_neon.h:34 26:3: error: argument must be a constant"
231 "-finstrument-functions-exclude-file-list=arm_neon.h", 239 "-finstrument-functions-exclude-file-list=arm_neon.h",
232 ] 240 ]
233 } 241 }
234 } 242 }
235 243
236 config("no_cygprofile_instrumentation") { 244 config("no_cygprofile_instrumentation") {
237 } 245 }
OLDNEW
« no previous file with comments | « build/config/BUILDCONFIG.gn ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698