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

Side by Side Diff: build/config/android/config.gni

Issue 2123753005: Creating new gn arg: enable_all_proguard_optimizations. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased again Created 4 years, 5 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.gn ('k') | build/config/android/rules.gni » ('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 # This file contains common system config stuff for the Android build. 5 # This file contains common system config stuff for the Android build.
6 6
7 if (is_android) { 7 if (is_android) {
8 assert(rebase_path("//", root_build_dir) == "../../", 8 assert(rebase_path("//", root_build_dir) == "../../",
9 "Android output directory must be nested 2 levels within src/ (" + 9 "Android output directory must be nested 2 levels within src/ (" +
10 "e.g.: out-gn/Debug). http://crbug.com/412935") 10 "e.g.: out-gn/Debug). http://crbug.com/412935")
(...skipping 106 matching lines...) Expand 10 before | Expand all | Expand 10 after
117 117
118 # EMMA filter string consisting of a list of inclusion/exclusion patterns 118 # EMMA filter string consisting of a list of inclusion/exclusion patterns
119 # separated with whitespace and/or comma. Only has effect if 119 # separated with whitespace and/or comma. Only has effect if
120 # emma_coverage==true 120 # emma_coverage==true
121 emma_filter = "" 121 emma_filter = ""
122 122
123 # Disables process isolation when building _incremental targets. 123 # Disables process isolation when building _incremental targets.
124 # Required for Android M+ due to SELinux policies (stronger sandboxing). 124 # Required for Android M+ due to SELinux policies (stronger sandboxing).
125 disable_incremental_isolated_processes = false 125 disable_incremental_isolated_processes = false
126 126
127 # Enables all ProGuard optimizations. These optimizations must not be
128 # enabled for instrumentation tests, since they cause code required by the
129 # tests to be removed.
130 # TODO(smaier): when buildbots get updated to set this flag, change the
131 # default to is_official_build
132 enable_all_proguard_optimizations = false
133
127 # Speed up incremental compiles by compiling only changed files. 134 # Speed up incremental compiles by compiling only changed files.
128 enable_incremental_javac = false 135 enable_incremental_javac = false
129 136
130 # Speed up dexing using dx --incremental. 137 # Speed up dexing using dx --incremental.
131 enable_incremental_dx = is_java_debug 138 enable_incremental_dx = is_java_debug
132 139
133 # Neither of these should ever be used for release builds since they are
134 # somewhat experimental and dx --incremental is known to not produce
135 # byte-for-byte identical output.
136 assert(!(enable_incremental_dx && !is_java_debug))
137 assert(!(enable_incremental_javac && !is_java_debug))
138
139 # Adds intrumentation to each function. Writes a file with the order that 140 # Adds intrumentation to each function. Writes a file with the order that
140 # functions are called at startup. 141 # functions are called at startup.
141 use_order_profiling = false 142 use_order_profiling = false
142 } 143 }
143 144
145 # Ensuring we never have a situation where we are asking to have debug java
146 # on alongside all ProGuard optimizations turned on, as these are mutually
147 # exclusive.
148 assert(!(is_java_debug && enable_all_proguard_optimizations))
149
150 # Neither of these should ever be used for release builds since they are
151 # somewhat experimental and dx --incremental is known to not produce
152 # byte-for-byte identical output.
153 assert(!(enable_incremental_dx && !is_java_debug))
154 assert(!(enable_incremental_javac && !is_java_debug))
155
144 # Host stuff ----------------------------------------------------------------- 156 # Host stuff -----------------------------------------------------------------
145 157
146 # Defines the name the Android build gives to the current host CPU 158 # Defines the name the Android build gives to the current host CPU
147 # architecture, which is different than the names GN uses. 159 # architecture, which is different than the names GN uses.
148 if (host_cpu == "x64") { 160 if (host_cpu == "x64") {
149 android_host_arch = "x86_64" 161 android_host_arch = "x86_64"
150 } else if (host_cpu == "x86") { 162 } else if (host_cpu == "x86") {
151 android_host_arch = "x86" 163 android_host_arch = "x86"
152 } else { 164 } else {
153 assert(false, "Need Android toolchain support for your build CPU arch.") 165 assert(false, "Need Android toolchain support for your build CPU arch.")
(...skipping 128 matching lines...) Expand 10 before | Expand all | Expand 10 after
282 } else if (current_cpu == "mips64el") { 294 } else if (current_cpu == "mips64el") {
283 android_app_abi = "mips64" 295 android_app_abi = "mips64"
284 } else { 296 } else {
285 assert(false, "Unknown Android ABI: " + current_cpu) 297 assert(false, "Unknown Android ABI: " + current_cpu)
286 } 298 }
287 299
288 if (android_libcpp_lib_dir == "") { 300 if (android_libcpp_lib_dir == "") {
289 android_libcpp_lib_dir = "${android_libcpp_root}/libs/${android_app_abi}" 301 android_libcpp_lib_dir = "${android_libcpp_root}/libs/${android_app_abi}"
290 } 302 }
291 } 303 }
OLDNEW
« no previous file with comments | « BUILD.gn ('k') | build/config/android/rules.gni » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698