Revision: 12930
Author:   [email protected]
Date:     Mon Nov 12 05:27:36 2012
Log: Fixed Code::FindCodeAgeSequence logic, removing a dead method on the way.

Review URL: https://codereview.chromium.org/11364177
http://code.google.com/p/v8/source/detail?r=12930

Modified:
 /branches/bleeding_edge/src/mips/assembler-mips.h
 /branches/bleeding_edge/src/objects.cc

=======================================
--- /branches/bleeding_edge/src/mips/assembler-mips.h Fri Nov 9 05:43:48 2012 +++ /branches/bleeding_edge/src/mips/assembler-mips.h Mon Nov 12 05:27:36 2012
@@ -524,9 +524,6 @@
   // upon destruction of the assembler.
   Assembler(Isolate* isolate, void* buffer, int buffer_size);
   ~Assembler();
-
-  // Dummy for cross platform compatibility.
-  void set_predictable_code_size(bool value) { }

   // GetCode emits any pending (non-emitted) code and fills the descriptor
   // desc. GetCode() is idempotent; it returns the same result if no other
=======================================
--- /branches/bleeding_edge/src/objects.cc      Mon Nov 12 03:08:34 2012
+++ /branches/bleeding_edge/src/objects.cc      Mon Nov 12 05:27:36 2012
@@ -8834,13 +8834,13 @@


 byte* Code::FindCodeAgeSequence() {
-  if (kind() != FUNCTION && kind() != OPTIMIZED_FUNCTION) return NULL;
-  if (strlen(FLAG_stop_at) == 0 &&
+  return FLAG_age_code &&
+      strlen(FLAG_stop_at) == 0 &&
       !ProfileEntryHookStub::HasEntryHook() &&
-      (kind() == FUNCTION && !has_debug_break_slots())) {
-    return FindPlatformCodeAgeSequence();
-  }
-  return NULL;
+      (kind() == OPTIMIZED_FUNCTION ||
+       (kind() == FUNCTION && !has_debug_break_slots()))
+      ? FindPlatformCodeAgeSequence()
+      : NULL;
 }


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

Reply via email to