http://codereview.chromium.org/6771045/diff/2001/src/compiler.cc
File src/compiler.cc (right):

http://codereview.chromium.org/6771045/diff/2001/src/compiler.cc#newcode700
src/compiler.cc:700: if (FLAG_lazy && allow_lazy) {
On 2011/04/01 09:45:43, Kevin Millikin wrote:
I think this whole conditional could be simpler:

if (FLAG_lazy && allow_lazy) {
   Handle<Code> code = info.isolate()->builtins()->LazyCompile();
   info.SetCode(code);
} else if ((V8::UseCrankshaft() && MakeCrankshaftCode(&info)) ||
            (!V8::UseCrankshaft() &&
FullCodeGenerator::MakeCode(&info))) {
   ASSERT(!info.code().is_null());
   scope_info = SerializedScopeInfo::Create(info.scope());
} else {
   return Handle<SharedFunctionInfo>::null();
}

Done.

http://codereview.chromium.org/6771045/

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

Reply via email to