Chromium Code Reviews| Index: src/ia32/macro-assembler-ia32.cc | 
| =================================================================== | 
| --- src/ia32/macro-assembler-ia32.cc (revision 6703) | 
| +++ src/ia32/macro-assembler-ia32.cc (working copy) | 
| @@ -604,11 +604,11 @@ | 
| ExternalReference new_space_allocation_limit = | 
| ExternalReference::new_space_allocation_limit_address(); | 
| - if (top_reg.is(result)) { | 
| - add(Operand(top_reg), Immediate(object_size)); | 
| - } else { | 
| - lea(top_reg, Operand(result, object_size)); | 
| + if (!top_reg.is(result)) { | 
| + mov(top_reg, result); | 
| } | 
| + add(Operand(top_reg), Immediate(object_size)); | 
| + j(carry, gc_required, not_taken); | 
| cmp(top_reg, Operand::StaticVariable(new_space_allocation_limit)); | 
| j(above, gc_required, not_taken); | 
| @@ -657,7 +657,12 @@ | 
| // Calculate new top and bail out if new space is exhausted. | 
| ExternalReference new_space_allocation_limit = | 
| ExternalReference::new_space_allocation_limit_address(); | 
| - lea(result_end, Operand(result, element_count, element_size, header_size)); | 
| + | 
| + // We assume that element_count*element_size + header_size does not | 
| + // overflow. | 
| + lea(result_end, Operand(element_count, element_size, header_size)); | 
| + add(result_end, Operand(result)); | 
| + j(carry, gc_required); | 
| cmp(result_end, Operand::StaticVariable(new_space_allocation_limit)); | 
| j(above, gc_required); | 
| @@ -702,6 +707,7 @@ | 
| mov(result_end, object_size); | 
| } | 
| add(result_end, Operand(result)); | 
| + j(carry, gc_required, not_taken); | 
| cmp(result_end, Operand::StaticVariable(new_space_allocation_limit)); | 
| j(above, gc_required, not_taken); | 
| @@ -1571,6 +1577,20 @@ | 
| } | 
| +void MacroAssembler::Ret(int bytes_dropped, Register scratch) { | 
| + if (is_uint16(bytes_dropped)) { | 
| + ret(bytes_dropped); | 
| + } else { | 
| + pop(scratch); | 
| + add(Operand(esp), Immediate(bytes_dropped)); | 
| + push(scratch); | 
| + ret(0); | 
| + } | 
| +} | 
| + | 
| + | 
| 
 
Rico
2011/02/10 06:58:20
Remove two blank lines
 
 | 
| + | 
| + | 
| void MacroAssembler::Drop(int stack_elements) { | 
| if (stack_elements > 0) { | 
| add(Operand(esp), Immediate(stack_elements * kPointerSize)); |