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

Unified Diff: BUILD.gn

Issue 2153753002: [gn] Set correct defaults for some gn args (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | build_overrides/v8.gni » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: BUILD.gn
diff --git a/BUILD.gn b/BUILD.gn
index 3491401da90516e209c4a60dd2240a77ee3190af..5174944b5449933d27f17fd8c1f2640b2759e622 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -26,7 +26,7 @@ declare_args() {
v8_deprecation_warnings = false
# Enable compiler warnings when using V8_DEPRECATE_SOON apis.
- v8_imminent_deprecation_warnings = false
+ v8_imminent_deprecation_warnings = ""
# Embeds the given script into the snapshot.
v8_embed_script = ""
@@ -38,7 +38,7 @@ declare_args() {
v8_enable_disassembler = false
# Sets -dENABLE_GDB_JIT_INTERFACE.
- v8_enable_gdbjit = false
+ v8_enable_gdbjit = ""
# Sets -dENABLE_HANDLE_ZAPPING.
v8_enable_handle_zapping = true
@@ -69,6 +69,23 @@ declare_args() {
v8_use_mips_abi_hardfloat = true
}
+# Set project-specific defaults for some args if not provided in args.gn. The
+# defaults can be set in the respective build_overrides files.
+if (v8_imminent_deprecation_warnings == "") {
+ if (defined(v8_imminent_deprecation_warnings_default)) {
Michael Achenbach 2016/07/15 11:33:17 Without guarding with the defined check, we'd need
+ v8_imminent_deprecation_warnings = v8_imminent_deprecation_warnings_default
+ } else {
+ v8_imminent_deprecation_warnings = false
+ }
+}
+if (v8_enable_gdbjit == "") {
+ if (defined(v8_enable_gdbjit_default)) {
+ v8_enable_gdbjit = v8_enable_gdbjit_default
+ } else {
+ v8_enable_gdbjit = false
+ }
+}
+
v8_random_seed = "314159265"
v8_toolset_for_shell = "host"
« no previous file with comments | « no previous file | build_overrides/v8.gni » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698