Reviewers: Kevin Millikin,

Description:
Allow OSR for closures that reference context slots.

Please review this at http://codereview.chromium.org/7812023/

SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/

Affected files:
  M     src/runtime-profiler.cc


Index: src/runtime-profiler.cc
===================================================================
--- src/runtime-profiler.cc     (revision 9063)
+++ src/runtime-profiler.cc     (working copy)
@@ -115,10 +115,8 @@
   }

   SharedFunctionInfo* shared = function->shared();
- // If the code is not optimizable or references context slots, don't try OSR. - if (!shared->code()->optimizable() | | !shared->allows_lazy_compilation()) {
-    return;
-  }
+  // If the code is not optimizable, don't try OSR.
+  if (!shared->code()->optimizable()) return;

   // We are not prepared to do OSR for a function that already has an
   // allocated arguments object.  The optimized code would bypass it for


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

Reply via email to