Reviewers: Michael Starzinger,

Message:
As discussed.

Description:
Move v8_optimized_debug default value to standalone.gypi

This is so that Chromium can have its own overridable default value without
colliding with ours.

Please review this at https://codereview.chromium.org/26317009/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files (+14, -14 lines):
  M build/standalone.gypi
  M build/toolchain.gypi


Index: build/standalone.gypi
diff --git a/build/standalone.gypi b/build/standalone.gypi
index 3d4f8049ad2352f9dea42fa338f98254bd05716e..4cb5e00bcc760fa9330bbca7114703f0899bd3b1 100644
--- a/build/standalone.gypi
+++ b/build/standalone.gypi
@@ -77,6 +77,20 @@
     # as errors.
     'v8_code%': 0,

+    # Speeds up Debug builds:
+    # 0 - Compiler optimizations off (debuggable) (default). This may
+    #     be 5x slower than Release (or worse).
+    # 1 - Turn on compiler optimizations. This may be hard or impossible to
+    #     debug. This may still be 2x slower than Release (or worse).
+    # 2 - Turn on optimizations, and also #undef DEBUG / #define NDEBUG
+    #     (but leave V8_ENABLE_CHECKS and most other assertions enabled.
+    #     This may cause some v8 tests to fail in the Debug configuration.
+    #     This roughly matches the performance of a Release build and can
+    #     be used by embedders that need to build their own code as debug
+    #     but don't want or need a debug version of V8. This should produce
+    #     near-release speeds.
+    'v8_optimized_debug%': 0,
+
     # Relative path to icu.gyp from this file.
     'icu_gyp_path': '../third_party/icu/icu.gyp',

Index: build/toolchain.gypi
diff --git a/build/toolchain.gypi b/build/toolchain.gypi
index c1066ebe94b7697968d211e5e98b6dc7ff747c9d..e1903f300fd757090f123a16f7456880b9779748 100644
--- a/build/toolchain.gypi
+++ b/build/toolchain.gypi
@@ -60,20 +60,6 @@

     'v8_enable_backtrace%': 0,

-    # Speeds up Debug builds:
-    # 0 - Compiler optimizations off (debuggable) (default). This may
-    #     be 5x slower than Release (or worse).
-    # 1 - Turn on compiler optimizations. This may be hard or impossible to
-    #     debug. This may still be 2x slower than Release (or worse).
-    # 2 - Turn on optimizations, and also #undef DEBUG / #define NDEBUG
-    #     (but leave V8_ENABLE_CHECKS and most other assertions enabled.
-    #     This may cause some v8 tests to fail in the Debug configuration.
-    #     This roughly matches the performance of a Release build and can
-    #     be used by embedders that need to build their own code as debug
-    #     but don't want or need a debug version of V8. This should produce
-    #     near-release speeds.
-    'v8_optimized_debug%': 0,
-
     # Enable profiling support. Only required on Windows.
     'v8_enable_prof%': 0,



--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to