LGTM. Sorry for not spotting this earlier. On Mon, Sep 22, 2008 at 5:54 PM, <[EMAIL PROTECTED]> wrote: > Reviewers: Erik Corry, Kasper Lund, > > Description: > Remove debugging code that snuck in with the ARM calling convention > changes. > Thanks to Erik Corry for spotting this. > > > Please review this at http://codereview.chromium.org/4015 > > Affected files: > M src/builtins-arm.cc > > > Index: src/builtins-arm.cc > =================================================================== > --- src/builtins-arm.cc (revision 355) > +++ src/builtins-arm.cc (working copy) > @@ -425,12 +425,6 @@ > // Eagerly check for stack-overflow before starting to push the arguments. > // r0: number of arguments > Label okay; > - { Label L; > - __ mov(r1, Operand(391864 << kSmiTagSize)); > - __ cmp(r0, r1); > - __ b(cc, &L); > - __ bind(&L); > - } > ExternalReference stack_guard_limit_address = > ExternalReference::address_of_stack_guard_limit(); > __ mov(r2, Operand(stack_guard_limit_address)); > > >
--~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
