LGTM
http://codereview.chromium.org/9769/diff/206/11 File src/codegen-ia32.cc (right): http://codereview.chromium.org/9769/diff/206/11#newcode158 Line 158: frame_->AllocateLocals(scope_->num_stack_slots()); Should AllocateLocals be called AllocateStackSlots now? http://codereview.chromium.org/9769/diff/206/8 File src/virtual-frame-ia32.cc (right): http://codereview.chromium.org/9769/diff/206/8#newcode43 Line 43: elements_(cgen->scope()->num_parameters() + 5), 5? http://codereview.chromium.org/9769/diff/206/9 File src/virtual-frame-ia32.h (right): http://codereview.chromium.org/9769/diff/206/9#newcode76 Line 76: void Adjust(int count); The name Adjust seems mighty generic. Perhaps AdjustHeight? http://codereview.chromium.org/9769/diff/206/9#newcode79 Line 79: void Forget(int count); ForgetElements? http://codereview.chromium.org/9769/diff/206/9#newcode108 Line 108: ASSERT(0 <= index && index < local_count_); 2 different asserts is nicer here if one is triggered. http://codereview.chromium.org/9769/diff/206/9#newcode150 Line 150: // emit code to effect the physical frame. Does not clobber any registers effect -> affect http://codereview.chromium.org/9769 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
