 Chromium Code Reviews
 Chromium Code Reviews Issue 1196193014:
  Do not add extra argument for new.target  (Closed) 
  Base URL: https://chromium.googlesource.com/v8/v8.git@master
    
  
    Issue 1196193014:
  Do not add extra argument for new.target  (Closed) 
  Base URL: https://chromium.googlesource.com/v8/v8.git@master| Index: src/ia32/builtins-ia32.cc | 
| diff --git a/src/ia32/builtins-ia32.cc b/src/ia32/builtins-ia32.cc | 
| index 429be9227aa907b88ee4aa2fdf17590486872084..da7ab51204200895c87bf797a25958949a7bc3d6 100644 | 
| --- a/src/ia32/builtins-ia32.cc | 
| +++ b/src/ia32/builtins-ia32.cc | 
| @@ -530,9 +530,6 @@ void Builtins::Generate_JSConstructStubForDerived(MacroAssembler* masm) { | 
| __ dec(ecx); | 
| __ j(greater_equal, &loop); | 
| - __ inc(eax); // Pushed new.target. | 
| - | 
| - | 
| // Handle step in. | 
| Label skip_step_in; | 
| ExternalReference debug_step_in_fp = | 
| @@ -556,7 +553,7 @@ void Builtins::Generate_JSConstructStubForDerived(MacroAssembler* masm) { | 
| // Restore context from the frame. | 
| __ mov(esi, Operand(ebp, StandardFrameConstants::kContextOffset)); | 
| - __ mov(ebx, Operand(esp, 0)); | 
| + __ mov(ebx, Operand(esp, kPointerSize)); | 
| 
adamk
2015/06/23 15:13:38
This could use a comment, like:
// Get arguments
 
Dmitry Lomov (no reviews)
2015/06/23 16:18:51
Done.
 | 
| } | 
| __ pop(ecx); // Return address. |