Chromium Code Reviews| Index: src/ia32/macro-assembler-ia32.cc |
| =================================================================== |
| --- src/ia32/macro-assembler-ia32.cc (revision 3513) |
| +++ src/ia32/macro-assembler-ia32.cc (working copy) |
| @@ -729,13 +729,13 @@ |
| cmp(result_end, Operand::StaticVariable(new_space_allocation_limit)); |
| j(above, gc_required, not_taken); |
| - // Update allocation top. |
| - UpdateAllocationTopHelper(result_end, scratch); |
| - |
| // Tag result if requested. |
| if ((flags & TAG_OBJECT) != 0) { |
| or_(Operand(result), Immediate(kHeapObjectTag)); |
| } |
| + |
| + // Update allocation top. |
| + UpdateAllocationTopHelper(result_end, scratch); |
|
sra1
2009/12/23 00:56:02
I assume the order was changed here generate instr
|
| } |
| @@ -759,13 +759,14 @@ |
| cmp(result_end, Operand::StaticVariable(new_space_allocation_limit)); |
| j(above, gc_required); |
| - // Update allocation top. |
| - UpdateAllocationTopHelper(result_end, scratch); |
| - |
| // Tag result if requested. |
| if ((flags & TAG_OBJECT) != 0) { |
| or_(Operand(result), Immediate(kHeapObjectTag)); |
| } |
| + |
| + // Update allocation top. |
| + UpdateAllocationTopHelper(result_end, scratch); |
| + |
| } |
| @@ -790,13 +791,13 @@ |
| cmp(result_end, Operand::StaticVariable(new_space_allocation_limit)); |
| j(above, gc_required, not_taken); |
| - // Update allocation top. |
| - UpdateAllocationTopHelper(result_end, scratch); |
| - |
| // Tag result if requested. |
| if ((flags & TAG_OBJECT) != 0) { |
| or_(Operand(result), Immediate(kHeapObjectTag)); |
| } |
| + |
| + // Update allocation top. |
| + UpdateAllocationTopHelper(result_end, scratch); |
| } |