| Index: src/builtins/ia32/builtins-ia32.cc
|
| diff --git a/src/builtins/ia32/builtins-ia32.cc b/src/builtins/ia32/builtins-ia32.cc
|
| index 8770d320bf07e4d8a6ce0aeee8e1c1542c3b52c5..b6c228361bb0fb0e3c96082754b8492b544a05db 100644
|
| --- a/src/builtins/ia32/builtins-ia32.cc
|
| +++ b/src/builtins/ia32/builtins-ia32.cc
|
| @@ -1858,39 +1858,17 @@ void Builtins::Generate_ReflectConstruct(MacroAssembler* masm) {
|
| // -- esp[4] : receiver (undefined)
|
| // -----------------------------------
|
|
|
| - // 2. Make sure the target is actually a constructor.
|
| - Label target_not_constructor;
|
| - __ JumpIfSmi(edi, &target_not_constructor, Label::kNear);
|
| - __ mov(ecx, FieldOperand(edi, HeapObject::kMapOffset));
|
| - __ test_b(FieldOperand(ecx, Map::kBitFieldOffset),
|
| - Immediate(1 << Map::kIsConstructor));
|
| - __ j(zero, &target_not_constructor, Label::kNear);
|
| + // 2. We don't need to check explicitly for constructor target here,
|
| + // since that's the first thing the Construct/ConstructWithArrayLike
|
| + // builtins will do.
|
|
|
| - // 3. Make sure the target is actually a constructor.
|
| - Label new_target_not_constructor;
|
| - __ JumpIfSmi(edx, &new_target_not_constructor, Label::kNear);
|
| - __ mov(ecx, FieldOperand(edx, HeapObject::kMapOffset));
|
| - __ test_b(FieldOperand(ecx, Map::kBitFieldOffset),
|
| - Immediate(1 << Map::kIsConstructor));
|
| - __ j(zero, &new_target_not_constructor, Label::kNear);
|
| + // 3. We don't need to check explicitly for constructor new.target here,
|
| + // since that's the second thing the Construct/ConstructWithArrayLike
|
| + // builtins will do.
|
|
|
| - // 4a. Construct the target with the given new.target and argumentsList.
|
| + // 4. Construct the target with the given new.target and argumentsList.
|
| __ Jump(masm->isolate()->builtins()->ConstructWithArrayLike(),
|
| RelocInfo::CODE_TARGET);
|
| -
|
| - // 4b. The target is not a constructor, throw an appropriate TypeError.
|
| - __ bind(&target_not_constructor);
|
| - {
|
| - __ mov(Operand(esp, kPointerSize), edi);
|
| - __ TailCallRuntime(Runtime::kThrowNotConstructor);
|
| - }
|
| -
|
| - // 4c. The new.target is not a constructor, throw an appropriate TypeError.
|
| - __ bind(&new_target_not_constructor);
|
| - {
|
| - __ mov(Operand(esp, kPointerSize), edx);
|
| - __ TailCallRuntime(Runtime::kThrowNotConstructor);
|
| - }
|
| }
|
|
|
| void Builtins::Generate_InternalArrayCode(MacroAssembler* masm) {
|
|
|