Revision: 9576
Author:   [email protected]
Date:     Tue Oct 11 02:28:06 2011
Log:      Remove some unused and unneeded flags.
Review URL: http://codereview.chromium.org/8228004
http://code.google.com/p/v8/source/detail?r=9576

Modified:
 /branches/bleeding_edge/benchmarks/spinning-balls/v.js
 /branches/bleeding_edge/src/arm/code-stubs-arm.cc
 /branches/bleeding_edge/src/flag-definitions.h
 /branches/bleeding_edge/src/ia32/code-stubs-ia32.cc
 /branches/bleeding_edge/src/mips/code-stubs-mips.cc
 /branches/bleeding_edge/src/spaces.h
 /branches/bleeding_edge/src/x64/code-stubs-x64.cc

=======================================
--- /branches/bleeding_edge/benchmarks/spinning-balls/v.js Wed Oct 5 06:40:49 2011 +++ /branches/bleeding_edge/benchmarks/spinning-balls/v.js Tue Oct 11 02:28:06 2011
@@ -41,6 +41,9 @@
         };
   } )();
 }
+
+
+

 var kNPoints = 8000;
 var kNModifications = 20;
@@ -54,10 +57,22 @@
 var dyingPoints = void 0;
 var scene = void 0;
 var renderingStartTime = void 0;
+var omitFrameDueToVisibilityChange = false;
 var scene = void 0;
 var pausePlot = void 0;
 var splayTree = void 0;

+handleVisibilityChange();
+
+function handleVisibilityChange() {
+  console.log("visibility change");
+  omitFrameDueToVisibilityChange = true;
+ document.addEventListener("webkitvisibilitychange", handleVisibilityChange);
+  document.addEventListener("msvisibilitychange", handleVisibilityChange);
+  document.addEventListener("mozvisibilitychange", handleVisibilityChange);
+  document.addEventListener("visibilitychange", handleVisibilityChange);
+}
+

 function Point(x, y, z, payload) {
   this.x = x;
@@ -353,9 +368,12 @@
   scene.draw();

   var renderingEndTime = Date.now();
-  var pause = renderingEndTime - renderingStartTime;
-  pausePlot.addPause(pause);
+  if (!omitFrameDueToVisibilityChange) {
+    var pause = renderingEndTime - renderingStartTime;
+    pausePlot.addPause(pause);
+  }
   renderingStartTime = renderingEndTime;
+  omitFrameDueToVisibilityChange = false;

   pausePlot.draw();

=======================================
--- /branches/bleeding_edge/src/arm/code-stubs-arm.cc Mon Oct 10 09:09:03 2011 +++ /branches/bleeding_edge/src/arm/code-stubs-arm.cc Tue Oct 11 02:28:06 2011
@@ -4426,10 +4426,6 @@
 #ifdef V8_INTERPRETED_REGEXP
   __ TailCallRuntime(Runtime::kRegExpExec, 4, 1);
 #else  // V8_INTERPRETED_REGEXP
-  if (!FLAG_regexp_entry_native) {
-    __ TailCallRuntime(Runtime::kRegExpExec, 4, 1);
-    return;
-  }

   // Stack frame on entry.
   //  sp[0]: last_match_info (expected JSArray)
=======================================
--- /branches/bleeding_edge/src/flag-definitions.h      Mon Oct 10 05:17:50 2011
+++ /branches/bleeding_edge/src/flag-definitions.h      Tue Oct 11 02:28:06 2011
@@ -300,9 +300,6 @@
 DEFINE_bool(canonicalize_object_literal_maps, true,
             "Canonicalize maps for object literals.")

-DEFINE_bool(use_big_map_space, true,
-            "Use big map space, but don't compact if it grew too big.")
-
 DEFINE_int(max_map_space_pages, MapSpace::kMaxMapPageIndex - 1,
"Maximum number of pages in map space which still allows to encode " "forwarding pointers. That's actually a constant, but it's useful "
@@ -338,7 +335,6 @@

 // Regexp
 DEFINE_bool(regexp_optimization, true, "generate optimized regexp code")
-DEFINE_bool(regexp_entry_native, true, "use native code to enter regexp")

 // Testing flags test/cctest/test-{flags,api,serialization}.cc
 DEFINE_bool(testing_bool_flag, true, "testing_bool_flag")
@@ -465,9 +461,6 @@

 DEFINE_bool(trace_isolates, false, "trace isolate state changes")

-DEFINE_bool(trace_live_byte_count, false,
-            "trace updates to page live byte count")
-
 // VM state
 DEFINE_bool(log_state_changes, false, "Log state changes.")

=======================================
--- /branches/bleeding_edge/src/ia32/code-stubs-ia32.cc Mon Oct 10 09:09:03 2011 +++ /branches/bleeding_edge/src/ia32/code-stubs-ia32.cc Tue Oct 11 02:28:06 2011
@@ -3384,10 +3384,6 @@
 #ifdef V8_INTERPRETED_REGEXP
   __ TailCallRuntime(Runtime::kRegExpExec, 4, 1);
 #else  // V8_INTERPRETED_REGEXP
-  if (!FLAG_regexp_entry_native) {
-    __ TailCallRuntime(Runtime::kRegExpExec, 4, 1);
-    return;
-  }

   // Stack frame on entry.
   //  esp[0]: return address
=======================================
--- /branches/bleeding_edge/src/mips/code-stubs-mips.cc Mon Oct 10 09:09:03 2011 +++ /branches/bleeding_edge/src/mips/code-stubs-mips.cc Tue Oct 11 02:28:06 2011
@@ -4471,10 +4471,6 @@
 #ifdef V8_INTERPRETED_REGEXP
   __ TailCallRuntime(Runtime::kRegExpExec, 4, 1);
 #else  // V8_INTERPRETED_REGEXP
-  if (!FLAG_regexp_entry_native) {
-    __ TailCallRuntime(Runtime::kRegExpExec, 4, 1);
-    return;
-  }

   // Stack frame on entry.
   //  sp[0]: last_match_info (expected JSArray)
=======================================
--- /branches/bleeding_edge/src/spaces.h        Thu Sep 29 05:27:31 2011
+++ /branches/bleeding_edge/src/spaces.h        Tue Oct 11 02:28:06 2011
@@ -452,7 +452,7 @@
   // Manage live byte count (count of bytes known to be live,
   // because they are marked black).
   void ResetLiveBytes() {
-    if (FLAG_trace_live_byte_count) {
+    if (FLAG_gc_verbose) {
       PrintF("ResetLiveBytes:%p:%x->0\n",
              static_cast<void*>(this), live_byte_count_);
     }
@@ -460,7 +460,7 @@
   }
   void IncrementLiveBytes(int by) {
     ASSERT_LE(static_cast<unsigned>(live_byte_count_), size_);
-    if (FLAG_trace_live_byte_count) {
+    if (FLAG_gc_verbose) {
       printf("UpdateLiveBytes:%p:%x%c=%x->%x\n",
              static_cast<void*>(this), live_byte_count_,
              ((by < 0) ? '-' : '+'), ((by < 0) ? -by : by),
=======================================
--- /branches/bleeding_edge/src/x64/code-stubs-x64.cc Mon Oct 10 09:18:20 2011 +++ /branches/bleeding_edge/src/x64/code-stubs-x64.cc Tue Oct 11 02:28:06 2011
@@ -2438,10 +2438,6 @@
 #ifdef V8_INTERPRETED_REGEXP
   __ TailCallRuntime(Runtime::kRegExpExec, 4, 1);
 #else  // V8_INTERPRETED_REGEXP
-  if (!FLAG_regexp_entry_native) {
-    __ TailCallRuntime(Runtime::kRegExpExec, 4, 1);
-    return;
-  }

   // Stack frame on entry.
   //  rsp[0]: return address

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to