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

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

Issue 1759603002: [debugger] Ensure at least one breakable position per function. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 4 years, 9 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
« no previous file with comments | « src/debug/debug.cc ('k') | src/full-codegen/arm64/full-codegen-arm64.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 #if V8_TARGET_ARCH_ARM 5 #if V8_TARGET_ARCH_ARM
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 278 matching lines...) Expand 10 before | Expand all | Expand 10 after
289 SetVar(arguments, r0, r1, r2); 289 SetVar(arguments, r0, r1, r2);
290 } 290 }
291 291
292 if (FLAG_trace) { 292 if (FLAG_trace) {
293 __ CallRuntime(Runtime::kTraceEnter); 293 __ CallRuntime(Runtime::kTraceEnter);
294 } 294 }
295 295
296 // Visit the declarations and body unless there is an illegal 296 // Visit the declarations and body unless there is an illegal
297 // redeclaration. 297 // redeclaration.
298 if (scope()->HasIllegalRedeclaration()) { 298 if (scope()->HasIllegalRedeclaration()) {
299 Comment cmnt(masm_, "[ Declarations"); 299 EmitIllegalRedeclaration();
300 VisitForEffect(scope()->GetIllegalRedeclaration());
301
302 } else { 300 } else {
303 PrepareForBailoutForId(BailoutId::FunctionEntry(), NO_REGISTERS); 301 PrepareForBailoutForId(BailoutId::FunctionEntry(), NO_REGISTERS);
304 { Comment cmnt(masm_, "[ Declarations"); 302 { Comment cmnt(masm_, "[ Declarations");
305 VisitDeclarations(scope()->declarations()); 303 VisitDeclarations(scope()->declarations());
306 } 304 }
307 305
308 // Assert that the declarations do not use ICs. Otherwise the debugger 306 // Assert that the declarations do not use ICs. Otherwise the debugger
309 // won't be able to redirect a PC at an IC to the correct IC in newly 307 // won't be able to redirect a PC at an IC to the correct IC in newly
310 // recompiled code. 308 // recompiled code.
311 DCHECK_EQ(0, ic_total_count_); 309 DCHECK_EQ(0, ic_total_count_);
(...skipping 3947 matching lines...) Expand 10 before | Expand all | Expand 10 after
4259 DCHECK(interrupt_address == 4257 DCHECK(interrupt_address ==
4260 isolate->builtins()->OsrAfterStackCheck()->entry()); 4258 isolate->builtins()->OsrAfterStackCheck()->entry());
4261 return OSR_AFTER_STACK_CHECK; 4259 return OSR_AFTER_STACK_CHECK;
4262 } 4260 }
4263 4261
4264 4262
4265 } // namespace internal 4263 } // namespace internal
4266 } // namespace v8 4264 } // namespace v8
4267 4265
4268 #endif // V8_TARGET_ARCH_ARM 4266 #endif // V8_TARGET_ARCH_ARM
OLDNEW
« no previous file with comments | « src/debug/debug.cc ('k') | src/full-codegen/arm64/full-codegen-arm64.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698