Reviewers: Kasper Lund, Description: Remove check for heap allocated locals from code flushing.
Since bleeding edge revision 5056 we no longer need to check that there are no heap allocated locals in the scope info, since these will be preserved after the scope info was moved to the SharedFunctionInfo class. Please review this at http://codereview.chromium.org/2931016/show SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/ Affected files: M src/heap.cc Index: src/heap.cc =================================================================== --- src/heap.cc (revision 5062) +++ src/heap.cc (working copy) @@ -2480,16 +2480,9 @@ ThreadManager::IterateArchivedThreads(&threadvisitor); if (threadvisitor.FoundCode()) return; - // Check that there are heap allocated locals in the scopeinfo. If - // there is, we are potentially using eval and need the scopeinfo - // for variable resolution. - if (ScopeInfo<>::HasHeapAllocatedLocals(function_info->scope_info())) - return; - HandleScope scope; // Compute the lazy compilable version of the code, clear the scope info. function_info->set_code(*ComputeLazyCompile(function_info->length())); - function_info->set_scope_info(ScopeInfo<>::EmptyHeapObject()); } -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
