| Index: src/arm/codegen-arm.cc
|
| ===================================================================
|
| --- src/arm/codegen-arm.cc (revision 6955)
|
| +++ src/arm/codegen-arm.cc (working copy)
|
| @@ -1588,7 +1588,7 @@
|
| }
|
|
|
|
|
| -void CodeGenerator::Comparison(Condition cc,
|
| +void CodeGenerator::Comparison(Condition cond,
|
| Expression* left,
|
| Expression* right,
|
| bool strict) {
|
| @@ -1602,7 +1602,7 @@
|
| // result : cc register
|
|
|
| // Strict only makes sense for equality comparisons.
|
| - ASSERT(!strict || cc == eq);
|
| + ASSERT(!strict || cond == eq);
|
|
|
| Register lhs;
|
| Register rhs;
|
| @@ -1613,8 +1613,8 @@
|
| // We load the top two stack positions into registers chosen by the virtual
|
| // frame. This should keep the register shuffling to a minimum.
|
| // Implement '>' and '<=' by reversal to obtain ECMA-262 conversion order.
|
| - if (cc == gt || cc == le) {
|
| - cc = ReverseCondition(cc);
|
| + if (cond == gt || cond == le) {
|
| + cond = ReverseCondition(cond);
|
| lhs_is_smi = frame_->KnownSmiAt(0);
|
| rhs_is_smi = frame_->KnownSmiAt(1);
|
| lhs = frame_->PopToRegister();
|
| @@ -1654,7 +1654,7 @@
|
| // Perform non-smi comparison by stub.
|
| // CompareStub takes arguments in r0 and r1, returns <0, >0 or 0 in r0.
|
| // We call with 0 args because there are 0 on the stack.
|
| - CompareStub stub(cc, strict, NO_SMI_COMPARE_IN_STUB, lhs, rhs);
|
| + CompareStub stub(cond, strict, NO_SMI_COMPARE_IN_STUB, lhs, rhs);
|
| frame_->CallStub(&stub, 0);
|
| __ cmp(r0, Operand(0, RelocInfo::NONE));
|
| exit.Jump();
|
| @@ -1666,7 +1666,7 @@
|
| __ cmp(lhs, Operand(rhs));
|
|
|
| exit.Bind();
|
| - cc_reg_ = cc;
|
| + cc_reg_ = cond;
|
| }
|
|
|
|
|
| @@ -1761,7 +1761,7 @@
|
| // sp[2]: applicand.
|
|
|
| // Check that the receiver really is a JavaScript object.
|
| - __ BranchOnSmi(receiver_reg, &build_args);
|
| + __ JumpIfSmi(receiver_reg, &build_args);
|
| // We allow all JSObjects including JSFunctions. As long as
|
| // JS_FUNCTION_TYPE is the last instance type and it is right
|
| // after LAST_JS_OBJECT_TYPE, we do not have to check the upper
|
| @@ -1773,7 +1773,7 @@
|
|
|
| // Check that applicand.apply is Function.prototype.apply.
|
| __ ldr(r0, MemOperand(sp, kPointerSize));
|
| - __ BranchOnSmi(r0, &build_args);
|
| + __ JumpIfSmi(r0, &build_args);
|
| __ CompareObjectType(r0, r1, r2, JS_FUNCTION_TYPE);
|
| __ b(ne, &build_args);
|
| Handle<Code> apply_code(
|
| @@ -1785,7 +1785,7 @@
|
|
|
| // Check that applicand is a function.
|
| __ ldr(r1, MemOperand(sp, 2 * kPointerSize));
|
| - __ BranchOnSmi(r1, &build_args);
|
| + __ JumpIfSmi(r1, &build_args);
|
| __ CompareObjectType(r1, r2, r3, JS_FUNCTION_TYPE);
|
| __ b(ne, &build_args);
|
|
|
| @@ -1885,8 +1885,8 @@
|
|
|
| void CodeGenerator::Branch(bool if_true, JumpTarget* target) {
|
| ASSERT(has_cc());
|
| - Condition cc = if_true ? cc_reg_ : NegateCondition(cc_reg_);
|
| - target->Branch(cc);
|
| + Condition cond = if_true ? cc_reg_ : NegateCondition(cc_reg_);
|
| + target->Branch(cond);
|
| cc_reg_ = al;
|
| }
|
|
|
| @@ -4622,8 +4622,8 @@
|
| ASSERT(runtime.entry_frame() == NULL);
|
| runtime.set_entry_frame(frame_);
|
|
|
| - __ BranchOnNotSmi(exponent, &exponent_nonsmi);
|
| - __ BranchOnNotSmi(base, &base_nonsmi);
|
| + __ JumpIfNotSmi(exponent, &exponent_nonsmi);
|
| + __ JumpIfNotSmi(base, &base_nonsmi);
|
|
|
| heap_number_map = r6;
|
| __ LoadRoot(heap_number_map, Heap::kHeapNumberMapRootIndex);
|
| @@ -5576,7 +5576,7 @@
|
| deferred->Branch(lt);
|
| __ ldrb(tmp2, FieldMemOperand(tmp1, Map::kBitFieldOffset));
|
| __ tst(tmp2, Operand(KeyedLoadIC::kSlowCaseBitFieldMask));
|
| - deferred->Branch(nz);
|
| + deferred->Branch(ne);
|
|
|
| // Check the object's elements are in fast case and writable.
|
| __ ldr(tmp1, FieldMemOperand(object, JSObject::kElementsOffset));
|
| @@ -5593,7 +5593,7 @@
|
| __ mov(tmp2, index1);
|
| __ orr(tmp2, tmp2, index2);
|
| __ tst(tmp2, Operand(kSmiTagMask));
|
| - deferred->Branch(nz);
|
| + deferred->Branch(ne);
|
|
|
| // Check that both indices are valid.
|
| __ ldr(tmp2, FieldMemOperand(object, JSArray::kLengthOffset));
|
| @@ -5854,15 +5854,11 @@
|
| frame_->EmitPush(r0);
|
|
|
| } else if (slot != NULL && slot->type() == Slot::LOOKUP) {
|
| - // lookup the context holding the named variable
|
| + // Delete from the context holding the named variable.
|
| frame_->EmitPush(cp);
|
| frame_->EmitPush(Operand(variable->name()));
|
| - frame_->CallRuntime(Runtime::kLookupContext, 2);
|
| - // r0: context
|
| + frame_->CallRuntime(Runtime::kDeleteContextSlot, 2);
|
| frame_->EmitPush(r0);
|
| - frame_->EmitPush(Operand(variable->name()));
|
| - frame_->InvokeBuiltin(Builtins::DELETE, CALL_JS, 2);
|
| - frame_->EmitPush(r0);
|
|
|
| } else {
|
| // Default: Result of deleting non-global, not dynamically
|
|
|