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

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

Issue 2684123002: Revert of Android: Use linker script to hide all non-JNI symbols (Closed)
Patch Set: Created 3 years, 10 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') | chrome/android/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 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 177 matching lines...) Expand 10 before | Expand all | Expand 10 after
188 ldflags += [ "-Wl,--warn-shared-textrel" ] 188 ldflags += [ "-Wl,--warn-shared-textrel" ]
189 } 189 }
190 } 190 }
191 191
192 config("executable_config") { 192 config("executable_config") {
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") {
199 ldflags = [ "-Wl,--version-script=" +
200 rebase_path("//build/android/android_no_jni_exports.lst") ]
201 }
202
198 config("hide_all_but_jni_onload") { 203 config("hide_all_but_jni_onload") {
199 ldflags = [ "-Wl,--version-script=" + rebase_path( 204 # TODO(agrieve): https://codereview.chromium.org/2633593004/
200 "//build/android/android_only_explicit_jni_exports.lst") ]
201 } 205 }
202 206
203 config("hide_all_but_jni") { 207 config("hide_all_but_jni") {
204 ldflags = [ "-Wl,--version-script=" + 208 # TODO(agrieve): https://codereview.chromium.org/2633593004/
205 rebase_path("//build/android/android_only_jni_exports.lst") ]
206 } 209 }
207 210
208 # Instrumentation ------------------------------------------------------------- 211 # Instrumentation -------------------------------------------------------------
209 # 212 #
210 # The BUILDCONFIG file sets the "default_cygprofile_instrumentation" config on 213 # The BUILDCONFIG file sets the "default_cygprofile_instrumentation" config on
211 # targets by default. You can override whether the cygprofile instrumentation is 214 # targets by default. You can override whether the cygprofile instrumentation is
212 # used on a per-target basis: 215 # used on a per-target basis:
213 # 216 #
214 # configs -= [ "//build/config/android:default_cygprofile_instrumentation" ] 217 # configs -= [ "//build/config/android:default_cygprofile_instrumentation" ]
215 # configs += [ "//build/config/android:no_cygprofile_instrumentation" ] 218 # configs += [ "//build/config/android:no_cygprofile_instrumentation" ]
(...skipping 17 matching lines...) Expand all
233 236
234 # Avoid errors with current NDK: 237 # Avoid errors with current NDK:
235 # "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"
236 "-finstrument-functions-exclude-file-list=arm_neon.h", 239 "-finstrument-functions-exclude-file-list=arm_neon.h",
237 ] 240 ]
238 } 241 }
239 } 242 }
240 243
241 config("no_cygprofile_instrumentation") { 244 config("no_cygprofile_instrumentation") {
242 } 245 }
OLDNEW
« no previous file with comments | « build/config/BUILDCONFIG.gn ('k') | chrome/android/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698