| Index: src/ia32/lithium-ia32.cc
|
| diff --git a/src/ia32/lithium-ia32.cc b/src/ia32/lithium-ia32.cc
|
| index 6b4db67aecd5501bd40069910fefa60cd5e2de94..930b7ed8784f5bde81c081566a3f0580e8522343 100644
|
| --- a/src/ia32/lithium-ia32.cc
|
| +++ b/src/ia32/lithium-ia32.cc
|
| @@ -679,15 +679,6 @@ LInstruction* LChunkBuilder::MarkAsCall(LInstruction* instr,
|
| instr->MarkAsCall();
|
| instr = AssignPointerMap(instr);
|
|
|
| - if (hinstr->HasObservableSideEffects()) {
|
| - ASSERT(hinstr->next()->IsSimulate());
|
| - HSimulate* sim = HSimulate::cast(hinstr->next());
|
| - ASSERT(instruction_pending_deoptimization_environment_ == NULL);
|
| - ASSERT(pending_deoptimization_ast_id_.IsNone());
|
| - instruction_pending_deoptimization_environment_ = instr;
|
| - pending_deoptimization_ast_id_ = sim->ast_id();
|
| - }
|
| -
|
| // If instruction does not have side-effects lazy deoptimization
|
| // after the call will try to deoptimize to the point before the call.
|
| // Thus we still need to attach environment to this call even if
|
| @@ -980,6 +971,26 @@ void LChunkBuilder::VisitInstruction(HInstruction* current) {
|
| chunk_->AddInstruction(clobber, current_block_);
|
| }
|
| chunk_->AddInstruction(instr, current_block_);
|
| +
|
| + if (instr->IsCall()) {
|
| + HValue* hydrogen_value_for_lazy_bailout = current;
|
| + LInstruction* instruction_needing_environment = NULL;
|
| + if (current->HasObservableSideEffects()) {
|
| + HSimulate* sim = HSimulate::cast(current->next());
|
| + instruction_needing_environment = instr;
|
| + sim->ReplayEnvironment(current_block_->last_environment());
|
| + hydrogen_value_for_lazy_bailout = sim;
|
| + }
|
| + LInstruction* bailout = AssignEnvironment(new(zone()) LLazyBailout());
|
| + bailout->set_hydrogen_value(hydrogen_value_for_lazy_bailout);
|
| + chunk_->AddInstruction(bailout, current_block_);
|
| + if (instruction_needing_environment != NULL) {
|
| + // Store the lazy deopt environment with the instruction if needed.
|
| + // Right now it is only used for LInstanceOfKnownGlobal.
|
| + instruction_needing_environment->
|
| + SetDeferredLazyDeoptimizationEnvironment(bailout->environment());
|
| + }
|
| + }
|
| }
|
| current_instruction_ = old_current;
|
| }
|
| @@ -1321,24 +1332,72 @@ LInstruction* LChunkBuilder::DoBitwise(HBitwise* instr) {
|
| }
|
|
|
|
|
| +LInstruction* LChunkBuilder::DoDivByPowerOf2I(HDiv* instr) {
|
| + ASSERT(instr->representation().IsSmiOrInteger32());
|
| + ASSERT(instr->left()->representation().Equals(instr->representation()));
|
| + ASSERT(instr->right()->representation().Equals(instr->representation()));
|
| + LOperand* dividend = UseRegister(instr->left());
|
| + int32_t divisor = instr->right()->GetInteger32Constant();
|
| + LInstruction* result = DefineAsRegister(new(zone()) LDivByPowerOf2I(
|
| + dividend, divisor));
|
| + if ((instr->CheckFlag(HValue::kBailoutOnMinusZero) && divisor < 0) ||
|
| + (instr->CheckFlag(HValue::kCanOverflow) && divisor == -1) ||
|
| + (!instr->CheckFlag(HInstruction::kAllUsesTruncatingToInt32) &&
|
| + divisor != 1 && divisor != -1)) {
|
| + result = AssignEnvironment(result);
|
| + }
|
| + return result;
|
| +}
|
| +
|
| +
|
| +LInstruction* LChunkBuilder::DoDivByConstI(HDiv* instr) {
|
| + ASSERT(instr->representation().IsInteger32());
|
| + ASSERT(instr->left()->representation().Equals(instr->representation()));
|
| + ASSERT(instr->right()->representation().Equals(instr->representation()));
|
| + LOperand* dividend = UseRegister(instr->left());
|
| + int32_t divisor = instr->right()->GetInteger32Constant();
|
| + LOperand* temp1 = FixedTemp(eax);
|
| + LOperand* temp2 = FixedTemp(edx);
|
| + LInstruction* result = DefineFixed(new(zone()) LDivByConstI(
|
| + dividend, divisor, temp1, temp2), edx);
|
| + if (divisor == 0 ||
|
| + (instr->CheckFlag(HValue::kBailoutOnMinusZero) && divisor < 0) ||
|
| + !instr->CheckFlag(HInstruction::kAllUsesTruncatingToInt32)) {
|
| + result = AssignEnvironment(result);
|
| + }
|
| + return result;
|
| +}
|
| +
|
| +
|
| +LInstruction* LChunkBuilder::DoDivI(HBinaryOperation* instr) {
|
| + ASSERT(instr->representation().IsSmiOrInteger32());
|
| + ASSERT(instr->left()->representation().Equals(instr->representation()));
|
| + ASSERT(instr->right()->representation().Equals(instr->representation()));
|
| + LOperand* dividend = UseFixed(instr->left(), eax);
|
| + LOperand* divisor = UseRegister(instr->right());
|
| + LOperand* temp = FixedTemp(edx);
|
| + LInstruction* result = DefineFixed(new(zone()) LDivI(
|
| + dividend, divisor, temp), eax);
|
| + if (instr->CheckFlag(HValue::kCanBeDivByZero) ||
|
| + instr->CheckFlag(HValue::kBailoutOnMinusZero) ||
|
| + instr->CheckFlag(HValue::kCanOverflow) ||
|
| + (!instr->IsMathFloorOfDiv() &&
|
| + !instr->CheckFlag(HValue::kAllUsesTruncatingToInt32))) {
|
| + result = AssignEnvironment(result);
|
| + }
|
| + return result;
|
| +}
|
| +
|
| +
|
| LInstruction* LChunkBuilder::DoDiv(HDiv* instr) {
|
| if (instr->representation().IsSmiOrInteger32()) {
|
| - ASSERT(instr->left()->representation().Equals(instr->representation()));
|
| - ASSERT(instr->right()->representation().Equals(instr->representation()));
|
| if (instr->RightIsPowerOf2()) {
|
| - ASSERT(!instr->CheckFlag(HValue::kCanBeDivByZero));
|
| - LOperand* value = UseRegister(instr->left());
|
| - LDivI* div =
|
| - new(zone()) LDivI(value, UseOrConstant(instr->right()), NULL);
|
| - return AssignEnvironment(DefineAsRegister(div));
|
| + return DoDivByPowerOf2I(instr);
|
| + } else if (instr->right()->IsConstant()) {
|
| + return DoDivByConstI(instr);
|
| + } else {
|
| + return DoDivI(instr);
|
| }
|
| - // The temporary operand is necessary to ensure that right is not allocated
|
| - // into edx.
|
| - LOperand* temp = FixedTemp(edx);
|
| - LOperand* dividend = UseFixed(instr->left(), eax);
|
| - LOperand* divisor = UseRegister(instr->right());
|
| - LDivI* result = new(zone()) LDivI(dividend, divisor, temp);
|
| - return AssignEnvironment(DefineFixed(result, eax));
|
| } else if (instr->representation().IsDouble()) {
|
| return DoArithmeticD(Token::DIV, instr);
|
| } else {
|
| @@ -1347,78 +1406,108 @@ LInstruction* LChunkBuilder::DoDiv(HDiv* instr) {
|
| }
|
|
|
|
|
| +LInstruction* LChunkBuilder::DoFlooringDivByPowerOf2I(HMathFloorOfDiv* instr) {
|
| + LOperand* dividend = UseRegisterAtStart(instr->left());
|
| + int32_t divisor = instr->right()->GetInteger32Constant();
|
| + LInstruction* result = DefineSameAsFirst(new(zone()) LFlooringDivByPowerOf2I(
|
| + dividend, divisor));
|
| + if ((instr->CheckFlag(HValue::kBailoutOnMinusZero) && divisor < 0) ||
|
| + (instr->CheckFlag(HValue::kLeftCanBeMinInt) && divisor == -1)) {
|
| + result = AssignEnvironment(result);
|
| + }
|
| + return result;
|
| +}
|
| +
|
| +
|
| +LInstruction* LChunkBuilder::DoFlooringDivByConstI(HMathFloorOfDiv* instr) {
|
| + ASSERT(instr->representation().IsInteger32());
|
| + ASSERT(instr->left()->representation().Equals(instr->representation()));
|
| + ASSERT(instr->right()->representation().Equals(instr->representation()));
|
| + LOperand* dividend = UseRegister(instr->left());
|
| + int32_t divisor = instr->right()->GetInteger32Constant();
|
| + LOperand* temp1 = FixedTemp(eax);
|
| + LOperand* temp2 = FixedTemp(edx);
|
| + LInstruction* result =
|
| + DefineFixed(new(zone()) LFlooringDivByConstI(dividend,
|
| + divisor,
|
| + temp1,
|
| + temp2),
|
| + edx);
|
| + bool can_deopt =
|
| + divisor == 0 ||
|
| + (instr->CheckFlag(HValue::kBailoutOnMinusZero) && divisor < 0);
|
| + return can_deopt ? AssignEnvironment(result) : result;
|
| +}
|
| +
|
| +
|
| LInstruction* LChunkBuilder::DoMathFloorOfDiv(HMathFloorOfDiv* instr) {
|
| - HValue* right = instr->right();
|
| - if (!right->IsConstant()) {
|
| - ASSERT(right->representation().IsInteger32());
|
| - // The temporary operand is necessary to ensure that right is not allocated
|
| - // into edx.
|
| - LOperand* temp = FixedTemp(edx);
|
| - LOperand* dividend = UseFixed(instr->left(), eax);
|
| - LOperand* divisor = UseRegister(instr->right());
|
| - LDivI* flooring_div = new(zone()) LDivI(dividend, divisor, temp);
|
| - return AssignEnvironment(DefineFixed(flooring_div, eax));
|
| - }
|
| -
|
| - ASSERT(right->IsConstant() && HConstant::cast(right)->HasInteger32Value());
|
| - LOperand* divisor = chunk_->DefineConstantOperand(HConstant::cast(right));
|
| - int32_t divisor_si = HConstant::cast(right)->Integer32Value();
|
| - if (divisor_si == 0) {
|
| - LOperand* dividend = UseRegister(instr->left());
|
| - return AssignEnvironment(DefineAsRegister(
|
| - new(zone()) LMathFloorOfDiv(dividend, divisor, NULL)));
|
| - } else if (IsPowerOf2(abs(divisor_si))) {
|
| - // use dividend as temp if divisor < 0 && divisor != -1
|
| - LOperand* dividend = divisor_si < -1 ? UseTempRegister(instr->left()) :
|
| - UseRegisterAtStart(instr->left());
|
| - LInstruction* result = DefineAsRegister(
|
| - new(zone()) LMathFloorOfDiv(dividend, divisor, NULL));
|
| - return divisor_si < 0 ? AssignEnvironment(result) : result;
|
| + if (instr->RightIsPowerOf2()) {
|
| + return DoFlooringDivByPowerOf2I(instr);
|
| + } else if (false && instr->right()->IsConstant()) {
|
| + return DoFlooringDivByConstI(instr); // TODO(svenpanne) Fix and re-enable.
|
| } else {
|
| - // needs edx:eax, plus a temp
|
| - LOperand* dividend = UseFixed(instr->left(), eax);
|
| - LOperand* temp = TempRegister();
|
| - LInstruction* result = DefineFixed(
|
| - new(zone()) LMathFloorOfDiv(dividend, divisor, temp), edx);
|
| - return divisor_si < 0 ? AssignEnvironment(result) : result;
|
| + return DoDivI(instr);
|
| }
|
| }
|
|
|
|
|
| +LInstruction* LChunkBuilder::DoModByPowerOf2I(HMod* instr) {
|
| + ASSERT(instr->representation().IsSmiOrInteger32());
|
| + ASSERT(instr->left()->representation().Equals(instr->representation()));
|
| + ASSERT(instr->right()->representation().Equals(instr->representation()));
|
| + LOperand* dividend = UseRegisterAtStart(instr->left());
|
| + int32_t divisor = instr->right()->GetInteger32Constant();
|
| + LInstruction* result = DefineSameAsFirst(new(zone()) LModByPowerOf2I(
|
| + dividend, divisor));
|
| + if (instr->CheckFlag(HValue::kBailoutOnMinusZero)) {
|
| + result = AssignEnvironment(result);
|
| + }
|
| + return result;
|
| +}
|
| +
|
| +
|
| +LInstruction* LChunkBuilder::DoModByConstI(HMod* instr) {
|
| + ASSERT(instr->representation().IsSmiOrInteger32());
|
| + ASSERT(instr->left()->representation().Equals(instr->representation()));
|
| + ASSERT(instr->right()->representation().Equals(instr->representation()));
|
| + LOperand* dividend = UseRegister(instr->left());
|
| + int32_t divisor = instr->right()->GetInteger32Constant();
|
| + LOperand* temp1 = FixedTemp(eax);
|
| + LOperand* temp2 = FixedTemp(edx);
|
| + LInstruction* result = DefineFixed(new(zone()) LModByConstI(
|
| + dividend, divisor, temp1, temp2), eax);
|
| + if (divisor == 0 || instr->CheckFlag(HValue::kBailoutOnMinusZero)) {
|
| + result = AssignEnvironment(result);
|
| + }
|
| + return result;
|
| +}
|
| +
|
| +
|
| +LInstruction* LChunkBuilder::DoModI(HMod* instr) {
|
| + ASSERT(instr->representation().IsSmiOrInteger32());
|
| + ASSERT(instr->left()->representation().Equals(instr->representation()));
|
| + ASSERT(instr->right()->representation().Equals(instr->representation()));
|
| + LOperand* dividend = UseFixed(instr->left(), eax);
|
| + LOperand* divisor = UseRegister(instr->right());
|
| + LOperand* temp = FixedTemp(edx);
|
| + LInstruction* result = DefineFixed(new(zone()) LModI(
|
| + dividend, divisor, temp), edx);
|
| + if (instr->CheckFlag(HValue::kCanBeDivByZero) ||
|
| + instr->CheckFlag(HValue::kBailoutOnMinusZero)) {
|
| + result = AssignEnvironment(result);
|
| + }
|
| + return result;
|
| +}
|
| +
|
| +
|
| LInstruction* LChunkBuilder::DoMod(HMod* instr) {
|
| - HValue* left = instr->left();
|
| - HValue* right = instr->right();
|
| if (instr->representation().IsSmiOrInteger32()) {
|
| - ASSERT(instr->left()->representation().Equals(instr->representation()));
|
| - ASSERT(instr->right()->representation().Equals(instr->representation()));
|
| -
|
| if (instr->RightIsPowerOf2()) {
|
| - ASSERT(!right->CanBeZero());
|
| - LModI* mod = new(zone()) LModI(UseRegisterAtStart(left),
|
| - UseOrConstant(right),
|
| - NULL);
|
| - LInstruction* result = DefineSameAsFirst(mod);
|
| - return (left->CanBeNegative() &&
|
| - instr->CheckFlag(HValue::kBailoutOnMinusZero))
|
| - ? AssignEnvironment(result)
|
| - : result;
|
| - return AssignEnvironment(DefineSameAsFirst(mod));
|
| + return DoModByPowerOf2I(instr);
|
| + } else if (instr->right()->IsConstant()) {
|
| + return DoModByConstI(instr);
|
| } else {
|
| - // The temporary operand is necessary to ensure that right is not
|
| - // allocated into edx.
|
| - LModI* mod = new(zone()) LModI(UseFixed(left, eax),
|
| - UseRegister(right),
|
| - FixedTemp(edx));
|
| - LInstruction* result = DefineFixed(mod, edx);
|
| - return (right->CanBeZero() ||
|
| - (left->RangeCanInclude(kMinInt) &&
|
| - right->RangeCanInclude(-1) &&
|
| - instr->CheckFlag(HValue::kBailoutOnMinusZero)) ||
|
| - (left->CanBeNegative() &&
|
| - instr->CanBeZero() &&
|
| - instr->CheckFlag(HValue::kBailoutOnMinusZero)))
|
| - ? AssignEnvironment(result)
|
| - : result;
|
| + return DoModI(instr);
|
| }
|
| } else if (instr->representation().IsDouble()) {
|
| return DoArithmeticD(Token::MOD, instr);
|
| @@ -1859,27 +1948,27 @@ LInstruction* LChunkBuilder::DoChange(HChange* instr) {
|
| if (to.IsTagged()) {
|
| HValue* val = instr->value();
|
| LOperand* value = UseRegister(val);
|
| - if (val->HasRange() && val->range()->IsInSmiRange()) {
|
| + if (!instr->CheckFlag(HValue::kCanOverflow)) {
|
| return DefineSameAsFirst(new(zone()) LSmiTag(value));
|
| } else if (val->CheckFlag(HInstruction::kUint32)) {
|
| - LOperand* temp = CpuFeatures::IsSupported(SSE2) ? FixedTemp(xmm1)
|
| - : NULL;
|
| - LNumberTagU* result = new(zone()) LNumberTagU(value, temp);
|
| + LOperand* temp1 = TempRegister();
|
| + LOperand* temp2 = CpuFeatures::IsSupported(SSE2) ? FixedTemp(xmm1)
|
| + : NULL;
|
| + LNumberTagU* result = new(zone()) LNumberTagU(value, temp1, temp2);
|
| return AssignEnvironment(AssignPointerMap(DefineSameAsFirst(result)));
|
| } else {
|
| - LNumberTagI* result = new(zone()) LNumberTagI(value);
|
| + LOperand* temp = TempRegister();
|
| + LNumberTagI* result = new(zone()) LNumberTagI(value, temp);
|
| return AssignEnvironment(AssignPointerMap(DefineSameAsFirst(result)));
|
| }
|
| } else if (to.IsSmi()) {
|
| HValue* val = instr->value();
|
| LOperand* value = UseRegister(val);
|
| - LInstruction* result = val->CheckFlag(HInstruction::kUint32)
|
| - ? DefineSameAsFirst(new(zone()) LUint32ToSmi(value))
|
| - : DefineSameAsFirst(new(zone()) LInteger32ToSmi(value));
|
| - if (val->HasRange() && val->range()->IsInSmiRange()) {
|
| - return result;
|
| + LInstruction* result = DefineSameAsFirst(new(zone()) LSmiTag(value));
|
| + if (instr->CheckFlag(HValue::kCanOverflow)) {
|
| + result = AssignEnvironment(result);
|
| }
|
| - return AssignEnvironment(result);
|
| + return result;
|
| } else {
|
| ASSERT(to.IsDouble());
|
| if (instr->value()->CheckFlag(HInstruction::kUint32)) {
|
| @@ -1972,6 +2061,20 @@ LInstruction* LChunkBuilder::DoClampToUint8(HClampToUint8* instr) {
|
| }
|
|
|
|
|
| +LInstruction* LChunkBuilder::DoDoubleBits(HDoubleBits* instr) {
|
| + HValue* value = instr->value();
|
| + ASSERT(value->representation().IsDouble());
|
| + return DefineAsRegister(new(zone()) LDoubleBits(UseRegister(value)));
|
| +}
|
| +
|
| +
|
| +LInstruction* LChunkBuilder::DoConstructDouble(HConstructDouble* instr) {
|
| + LOperand* lo = UseRegister(instr->lo());
|
| + LOperand* hi = UseRegister(instr->hi());
|
| + return DefineAsRegister(new(zone()) LConstructDouble(hi, lo));
|
| +}
|
| +
|
| +
|
| LInstruction* LChunkBuilder::DoReturn(HReturn* instr) {
|
| LOperand* context = info()->IsStub() ? UseFixed(instr->context(), esi) : NULL;
|
| LOperand* parameter_count = UseRegisterOrConstant(instr->parameter_count());
|
| @@ -2271,7 +2374,7 @@ LInstruction* LChunkBuilder::DoStoreNamedField(HStoreNamedField* instr) {
|
|
|
| bool can_be_constant = instr->value()->IsConstant() &&
|
| HConstant::cast(instr->value())->NotInNewSpace() &&
|
| - !(FLAG_track_double_fields && instr->field_representation().IsDouble());
|
| + !instr->field_representation().IsDouble();
|
|
|
| LOperand* val;
|
| if (instr->field_representation().IsInteger8() ||
|
| @@ -2283,10 +2386,9 @@ LInstruction* LChunkBuilder::DoStoreNamedField(HStoreNamedField* instr) {
|
| val = UseTempRegister(instr->value());
|
| } else if (can_be_constant) {
|
| val = UseRegisterOrConstant(instr->value());
|
| - } else if (FLAG_track_fields && instr->field_representation().IsSmi()) {
|
| + } else if (instr->field_representation().IsSmi()) {
|
| val = UseTempRegister(instr->value());
|
| - } else if (FLAG_track_double_fields &&
|
| - instr->field_representation().IsDouble()) {
|
| + } else if (instr->field_representation().IsDouble()) {
|
| val = UseRegisterAtStart(instr->value());
|
| } else {
|
| val = UseRegister(instr->value());
|
| @@ -2302,8 +2404,7 @@ LInstruction* LChunkBuilder::DoStoreNamedField(HStoreNamedField* instr) {
|
|
|
| LStoreNamedField* result =
|
| new(zone()) LStoreNamedField(obj, val, temp, temp_map);
|
| - if (FLAG_track_heap_object_fields &&
|
| - instr->field_representation().IsHeapObject()) {
|
| + if (instr->field_representation().IsHeapObject()) {
|
| if (!instr->value()->type().IsHeapObject()) {
|
| return AssignEnvironment(result);
|
| }
|
| @@ -2494,22 +2595,6 @@ LInstruction* LChunkBuilder::DoIsConstructCallAndBranch(
|
|
|
| LInstruction* LChunkBuilder::DoSimulate(HSimulate* instr) {
|
| instr->ReplayEnvironment(current_block_->last_environment());
|
| -
|
| - // If there is an instruction pending deoptimization environment create a
|
| - // lazy bailout instruction to capture the environment.
|
| - if (!pending_deoptimization_ast_id_.IsNone()) {
|
| - ASSERT(pending_deoptimization_ast_id_ == instr->ast_id());
|
| - LLazyBailout* lazy_bailout = new(zone()) LLazyBailout;
|
| - LInstruction* result = AssignEnvironment(lazy_bailout);
|
| - // Store the lazy deopt environment with the instruction if needed. Right
|
| - // now it is only used for LInstanceOfKnownGlobal.
|
| - instruction_pending_deoptimization_environment_->
|
| - SetDeferredLazyDeoptimizationEnvironment(result->environment());
|
| - instruction_pending_deoptimization_environment_ = NULL;
|
| - pending_deoptimization_ast_id_ = BailoutId::None();
|
| - return result;
|
| - }
|
| -
|
| return NULL;
|
| }
|
|
|
|
|