LGTM.
http://codereview.chromium.org/115347/diff/1/8 File src/compiler.cc (right): http://codereview.chromium.org/115347/diff/1/8#newcode104 Line 104: CompilationZoneScope zone_scope(DELETE_ON_EXIT); Do you ever have CompilationZoneScope that aren't DELETE_ON_EXIT? You could remove the constructor parameter... http://codereview.chromium.org/115347/diff/1/12 File src/frame-element.h (right): http://codereview.chromium.org/115347/diff/1/12#newcode145 Line 145: StaticType static_type() { return StaticType(StaticTypeField::decode(value_)); } Line too long? http://codereview.chromium.org/115347 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
