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

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

Issue 288213002: Refactor MacroAssembler::Prologue. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: addressed comment Created 6 years, 7 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | src/arm/lithium-codegen-arm.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 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 #include "v8.h" 5 #include "v8.h"
6 6
7 #if V8_TARGET_ARCH_ARM 7 #if V8_TARGET_ARCH_ARM
8 8
9 #include "code-stubs.h" 9 #include "code-stubs.h"
10 #include "codegen.h" 10 #include "codegen.h"
(...skipping 153 matching lines...) Expand 10 before | Expand all | Expand 10 after
164 164
165 __ bind(&ok); 165 __ bind(&ok);
166 } 166 }
167 167
168 // Open a frame scope to indicate that there is a frame on the stack. The 168 // Open a frame scope to indicate that there is a frame on the stack. The
169 // MANUAL indicates that the scope shouldn't actually generate code to set up 169 // MANUAL indicates that the scope shouldn't actually generate code to set up
170 // the frame (that is done below). 170 // the frame (that is done below).
171 FrameScope frame_scope(masm_, StackFrame::MANUAL); 171 FrameScope frame_scope(masm_, StackFrame::MANUAL);
172 172
173 info->set_prologue_offset(masm_->pc_offset()); 173 info->set_prologue_offset(masm_->pc_offset());
174 ASSERT(!info->IsStub()); 174 __ Prologue(info->IsCodePreAgingActive());
175 __ Prologue(info);
176 info->AddNoFrameRange(0, masm_->pc_offset()); 175 info->AddNoFrameRange(0, masm_->pc_offset());
177 176
178 { Comment cmnt(masm_, "[ Allocate locals"); 177 { Comment cmnt(masm_, "[ Allocate locals");
179 int locals_count = info->scope()->num_stack_slots(); 178 int locals_count = info->scope()->num_stack_slots();
180 // Generators allocate locals, if any, in context slots. 179 // Generators allocate locals, if any, in context slots.
181 ASSERT(!info->function()->is_generator() || locals_count == 0); 180 ASSERT(!info->function()->is_generator() || locals_count == 0);
182 if (locals_count > 0) { 181 if (locals_count > 0) {
183 if (locals_count >= 128) { 182 if (locals_count >= 128) {
184 EmitStackCheck(masm_, r2, locals_count, r9); 183 EmitStackCheck(masm_, r2, locals_count, r9);
185 } 184 }
(...skipping 4636 matching lines...) Expand 10 before | Expand all | Expand 10 after
4822 4821
4823 ASSERT(interrupt_address == 4822 ASSERT(interrupt_address ==
4824 isolate->builtins()->OsrAfterStackCheck()->entry()); 4823 isolate->builtins()->OsrAfterStackCheck()->entry());
4825 return OSR_AFTER_STACK_CHECK; 4824 return OSR_AFTER_STACK_CHECK;
4826 } 4825 }
4827 4826
4828 4827
4829 } } // namespace v8::internal 4828 } } // namespace v8::internal
4830 4829
4831 #endif // V8_TARGET_ARCH_ARM 4830 #endif // V8_TARGET_ARCH_ARM
OLDNEW
« no previous file with comments | « no previous file | src/arm/lithium-codegen-arm.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698