Revision: 5066 Author: [email protected] Date: Wed Jul 14 02:45:08 2010 Log: 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. Review URL: http://codereview.chromium.org/2931016 http://code.google.com/p/v8/source/detail?r=5066 Modified: /branches/bleeding_edge/src/heap.cc ======================================= --- /branches/bleeding_edge/src/heap.cc Tue Jul 13 06:06:33 2010 +++ /branches/bleeding_edge/src/heap.cc Wed Jul 14 02:45:08 2010 @@ -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; - + // Compute the lazy compilable version of the code. 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
