Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(213)

Side by Side Diff: src/full-codegen/ppc/full-codegen-ppc.cc

Issue 2276923002: Remove the rest_parameter(int*) variant, use rest_parameter() instead. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Ports. Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 the V8 project authors. All rights reserved. 1 // Copyright 2014 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #if V8_TARGET_ARCH_PPC 5 #if V8_TARGET_ARCH_PPC
6 6
7 #include "src/ast/scopes.h" 7 #include "src/ast/scopes.h"
8 #include "src/code-factory.h" 8 #include "src/code-factory.h"
9 #include "src/code-stubs.h" 9 #include "src/code-stubs.h"
10 #include "src/codegen.h" 10 #include "src/codegen.h"
(...skipping 241 matching lines...) Expand 10 before | Expand all | Expand 10 after
252 } 252 }
253 253
254 // Possibly set up a local binding to the new target value. 254 // Possibly set up a local binding to the new target value.
255 Variable* new_target_var = info->scope()->new_target_var(); 255 Variable* new_target_var = info->scope()->new_target_var();
256 if (new_target_var != nullptr) { 256 if (new_target_var != nullptr) {
257 Comment cmnt(masm_, "[ new.target"); 257 Comment cmnt(masm_, "[ new.target");
258 SetVar(new_target_var, r6, r3, r5); 258 SetVar(new_target_var, r6, r3, r5);
259 } 259 }
260 260
261 // Possibly allocate RestParameters 261 // Possibly allocate RestParameters
262 int rest_index; 262 Variable* rest_param = info->scope()->rest_parameter();
263 Variable* rest_param = info->scope()->rest_parameter(&rest_index); 263 if (rest_param != nullptr) {
264 if (rest_param) {
265 Comment cmnt(masm_, "[ Allocate rest parameter array"); 264 Comment cmnt(masm_, "[ Allocate rest parameter array");
266 if (!function_in_register_r4) { 265 if (!function_in_register_r4) {
267 __ LoadP(r4, MemOperand(fp, JavaScriptFrameConstants::kFunctionOffset)); 266 __ LoadP(r4, MemOperand(fp, JavaScriptFrameConstants::kFunctionOffset));
268 } 267 }
269 FastNewRestParameterStub stub(isolate()); 268 FastNewRestParameterStub stub(isolate());
270 __ CallStub(&stub); 269 __ CallStub(&stub);
271 function_in_register_r4 = false; 270 function_in_register_r4 = false;
272 SetVar(rest_param, r3, r4, r5); 271 SetVar(rest_param, r3, r4, r5);
273 } 272 }
274 273
(...skipping 3462 matching lines...) Expand 10 before | Expand all | Expand 10 after
3737 3736
3738 DCHECK(Assembler::IsCrSet(Assembler::instr_at(cmp_address))); 3737 DCHECK(Assembler::IsCrSet(Assembler::instr_at(cmp_address)));
3739 3738
3740 DCHECK(interrupt_address == 3739 DCHECK(interrupt_address ==
3741 isolate->builtins()->OnStackReplacement()->entry()); 3740 isolate->builtins()->OnStackReplacement()->entry());
3742 return ON_STACK_REPLACEMENT; 3741 return ON_STACK_REPLACEMENT;
3743 } 3742 }
3744 } // namespace internal 3743 } // namespace internal
3745 } // namespace v8 3744 } // namespace v8
3746 #endif // V8_TARGET_ARCH_PPC 3745 #endif // V8_TARGET_ARCH_PPC
OLDNEW
« no previous file with comments | « src/full-codegen/mips64/full-codegen-mips64.cc ('k') | src/full-codegen/s390/full-codegen-s390.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698