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

Unified Diff: BUILD.gn

Issue 2625393003: [build] Add more v8 build options and fix existing ones. (Closed)
Patch Set: Rename v8_object_print to v8_enable_object_print 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | 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 58bdb6b6531c57afdfcd02c606e5c8e9584a604e..b80a756c493d45d8be4e504956342aebfd3a45e5 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -21,7 +21,7 @@ declare_args() {
v8_android_log_stdout = false
# Sets -DVERIFY_HEAP.
- v8_enable_verify_heap = false
+ v8_enable_verify_heap = ""
# Sets -DVERIFY_PREDICTABLE
v8_enable_verify_predictable = false
@@ -52,7 +52,13 @@ declare_args() {
v8_interpreted_regexp = false
# Sets -dOBJECT_PRINT.
- v8_object_print = ""
+ v8_enable_object_print = ""
Michael Achenbach 2017/01/13 16:25:01 ok as it is consistent with the other flags now. B
+
+ # Sets -dTRACE_MAPS.
+ v8_enable_trace_maps = ""
+
+ # Sets -dV8_ENABLE_CHECKS.
+ v8_enable_v8_checks = ""
# With post mortem support enabled, metadata is embedded into libv8 that
# describes various parameters of the VM for use by debuggers. See
@@ -90,11 +96,20 @@ if (v8_enable_gdbjit == "") {
}
# Derived defaults.
-if (v8_object_print == "") {
- v8_object_print = is_debug && !v8_optimized_debug
+if (v8_enable_verify_heap == "") {
+ v8_enable_verify_heap = is_debug
+}
+if (v8_enable_object_print == "") {
+ v8_enable_object_print = is_debug
}
if (v8_enable_disassembler == "") {
- v8_enable_disassembler = is_debug && !v8_optimized_debug
+ v8_enable_disassembler = is_debug
+}
+if (v8_enable_trace_maps == "") {
+ v8_enable_trace_maps = is_debug
+}
+if (v8_enable_v8_checks == "") {
+ v8_enable_v8_checks = is_debug
}
# Specifies if the target build is a simulator build. Comparing target cpu
@@ -180,7 +195,7 @@ config("features") {
if (v8_enable_gdbjit) {
defines += [ "ENABLE_GDB_JIT_INTERFACE" ]
}
- if (v8_object_print) {
+ if (v8_enable_object_print) {
defines += [ "OBJECT_PRINT" ]
}
if (v8_enable_verify_heap) {
@@ -189,6 +204,12 @@ config("features") {
if (v8_enable_verify_predictable) {
defines += [ "VERIFY_PREDICTABLE" ]
}
+ if (v8_enable_trace_maps) {
+ defines += [ "TRACE_MAPS" ]
+ }
+ if (v8_enable_v8_checks) {
+ defines += [ "V8_ENABLE_CHECKS" ]
+ }
if (v8_interpreted_regexp) {
defines += [ "V8_INTERPRETED_REGEXP" ]
}
@@ -352,15 +373,7 @@ config("toolchain") {
ldflags += [ "-rdynamic" ]
}
- # TODO(jochen): Add support for different debug optimization levels.
- defines += [
- "ENABLE_DISASSEMBLER",
- "V8_ENABLE_CHECKS",
- "OBJECT_PRINT",
- "VERIFY_HEAP",
- "DEBUG",
- "TRACE_MAPS",
- ]
+ defines += [ "DEBUG" ]
if (v8_enable_slow_dchecks) {
defines += [ "ENABLE_SLOW_DCHECKS" ]
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698