Index: src/code-stubs.cc |
diff --git a/src/code-stubs.cc b/src/code-stubs.cc |
index 06b7afcaeb28cd90b182e374225ab9a66dfa030a..f55c21a926a53be86587199fbea138abe29ddb72 100644 |
--- a/src/code-stubs.cc |
+++ b/src/code-stubs.cc |
@@ -1491,13 +1491,32 @@ compiler::Node* IncStub::Generate(CodeStubAssembler* assembler, |
assembler->Bind(&if_valuenotnumber); |
{ |
- // Convert to a Number first and try again. |
- Callable callable = |
- CodeFactory::NonNumberToNumber(assembler->isolate()); |
- var_type_feedback.Bind( |
- assembler->Int32Constant(BinaryOperationFeedback::kAny)); |
- value_var.Bind(assembler->CallStub(callable, context, value)); |
- assembler->Goto(&start); |
+ Label if_valueisoddball(assembler), if_valuenotoddball(assembler); |
+ Node* instance_type = assembler->LoadMapInstanceType(value_map); |
+ Node* is_oddball = assembler->Word32Equal( |
+ instance_type, assembler->Int32Constant(ODDBALL_TYPE)); |
+ assembler->Branch(is_oddball, &if_valueisoddball, &if_valuenotoddball); |
+ |
+ assembler->Bind(&if_valueisoddball); |
+ { |
+ // Convert Oddball to Number and check again. |
+ value_var.Bind( |
+ assembler->LoadObjectField(value, Oddball::kToNumberOffset)); |
+ var_type_feedback.Bind(assembler->Int32Constant( |
Leszek Swirski
2016/10/19 08:42:48
Because of the loop, this should possibly be an "o
mythria
2016/10/19 11:26:37
As I explained offline, we don't need it here beca
|
+ BinaryOperationFeedback::kNumberOrOddball)); |
+ assembler->Goto(&start); |
+ } |
+ |
+ assembler->Bind(&if_valuenotoddball); |
+ { |
+ // Convert to a Number first and try again. |
+ Callable callable = |
+ CodeFactory::NonNumberToNumber(assembler->isolate()); |
+ var_type_feedback.Bind( |
+ assembler->Int32Constant(BinaryOperationFeedback::kAny)); |
+ value_var.Bind(assembler->CallStub(callable, context, value)); |
+ assembler->Goto(&start); |
+ } |
} |
} |
} |
@@ -1601,13 +1620,32 @@ compiler::Node* DecStub::Generate(CodeStubAssembler* assembler, |
assembler->Bind(&if_valuenotnumber); |
{ |
- // Convert to a Number first and try again. |
- Callable callable = |
- CodeFactory::NonNumberToNumber(assembler->isolate()); |
- var_type_feedback.Bind( |
- assembler->Int32Constant(BinaryOperationFeedback::kAny)); |
- value_var.Bind(assembler->CallStub(callable, context, value)); |
- assembler->Goto(&start); |
+ Label if_valueisoddball(assembler), if_valuenotoddball(assembler); |
+ Node* instance_type = assembler->LoadMapInstanceType(value_map); |
+ Node* is_oddball = assembler->Word32Equal( |
+ instance_type, assembler->Int32Constant(ODDBALL_TYPE)); |
+ assembler->Branch(is_oddball, &if_valueisoddball, &if_valuenotoddball); |
+ |
+ assembler->Bind(&if_valueisoddball); |
+ { |
+ // Convert Oddball to Number and check again. |
+ value_var.Bind( |
+ assembler->LoadObjectField(value, Oddball::kToNumberOffset)); |
+ var_type_feedback.Bind(assembler->Int32Constant( |
+ BinaryOperationFeedback::kNumberOrOddball)); |
+ assembler->Goto(&start); |
+ } |
+ |
+ assembler->Bind(&if_valuenotoddball); |
+ { |
+ // Convert to a Number first and try again. |
+ Callable callable = |
+ CodeFactory::NonNumberToNumber(assembler->isolate()); |
+ var_type_feedback.Bind( |
+ assembler->Int32Constant(BinaryOperationFeedback::kAny)); |
+ value_var.Bind(assembler->CallStub(callable, context, value)); |
+ assembler->Goto(&start); |
+ } |
} |
} |
} |