Reviewers: Mads Ager,
Description:
Temporarily allocate all function contexts in old space and
force the system to use the full compiler to momentarily get
better test coverage.
This change will be reverted once the unit tests have finished
running.
Please review this at http://codereview.chromium.org/552257
SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/
Affected files:
M src/flag-definitions.h
M src/heap.cc
Index: src/heap.cc
===================================================================
--- src/heap.cc (revision 3756)
+++ src/heap.cc (working copy)
@@ -3013,7 +3013,7 @@
Object* Heap::AllocateFunctionContext(int length, JSFunction* function) {
ASSERT(length >= Context::MIN_CONTEXT_SLOTS);
- Object* result = Heap::AllocateFixedArray(length);
+ Object* result = Heap::AllocateFixedArray(length, TENURED);
if (result->IsFailure()) return result;
Context* context = reinterpret_cast<Context*>(result);
context->set_map(context_map());
Index: src/flag-definitions.h
===================================================================
--- src/flag-definitions.h (revision 3756)
+++ src/flag-definitions.h (working copy)
@@ -145,7 +145,7 @@
"minimum length for automatic enable preparsing")
DEFINE_bool(full_compiler, true, "enable dedicated backend for run-once
code")
DEFINE_bool(fast_compiler, false, "enable speculative optimizing backend")
-DEFINE_bool(always_full_compiler, false,
+DEFINE_bool(always_full_compiler, true,
"try to use the dedicated run-once backend for all code")
DEFINE_bool(always_fast_compiler, false,
"try to use the speculative optimizing backend for all code")
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev