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

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

Issue 2650193002: [debugger] remove debugger statement support from FCG/CS. (Closed)
Patch Set: Created 3 years, 11 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/ia32/macro-assembler-ia32.h » ('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 "src/full-codegen/full-codegen.h" 5 #include "src/full-codegen/full-codegen.h"
6 6
7 #include "src/ast/ast-numbering.h" 7 #include "src/ast/ast-numbering.h"
8 #include "src/ast/ast.h" 8 #include "src/ast/ast.h"
9 #include "src/ast/prettyprinter.h" 9 #include "src/ast/prettyprinter.h"
10 #include "src/ast/scopes.h" 10 #include "src/ast/scopes.h"
(...skipping 1210 matching lines...) Expand 10 before | Expand all | Expand 10 after
1221 } 1221 }
1222 1222
1223 1223
1224 void FullCodeGenerator::VisitTryFinallyStatement(TryFinallyStatement* stmt) { 1224 void FullCodeGenerator::VisitTryFinallyStatement(TryFinallyStatement* stmt) {
1225 // Exception handling is not supported. 1225 // Exception handling is not supported.
1226 UNREACHABLE(); 1226 UNREACHABLE();
1227 } 1227 }
1228 1228
1229 1229
1230 void FullCodeGenerator::VisitDebuggerStatement(DebuggerStatement* stmt) { 1230 void FullCodeGenerator::VisitDebuggerStatement(DebuggerStatement* stmt) {
1231 Comment cmnt(masm_, "[ DebuggerStatement"); 1231 // Debugger statement is not supported.
1232 SetStatementPosition(stmt); 1232 UNREACHABLE();
1233
1234 __ DebugBreak();
1235 // Ignore the return value.
1236
1237 PrepareForBailoutForId(stmt->DebugBreakId(), BailoutState::NO_REGISTERS);
Michael Starzinger 2017/01/24 13:10:24 suggestion: We can now also remove DebuggerStateme
1238 } 1233 }
1239 1234
1240 1235
1241 void FullCodeGenerator::VisitCaseClause(CaseClause* clause) { 1236 void FullCodeGenerator::VisitCaseClause(CaseClause* clause) {
1242 UNREACHABLE(); 1237 UNREACHABLE();
1243 } 1238 }
1244 1239
1245 1240
1246 void FullCodeGenerator::VisitConditional(Conditional* expr) { 1241 void FullCodeGenerator::VisitConditional(Conditional* expr) {
1247 Comment cmnt(masm_, "[ Conditional"); 1242 Comment cmnt(masm_, "[ Conditional");
(...skipping 355 matching lines...) Expand 10 before | Expand all | Expand 10 after
1603 return info_->has_simple_parameters(); 1598 return info_->has_simple_parameters();
1604 } 1599 }
1605 1600
1606 FunctionLiteral* FullCodeGenerator::literal() const { return info_->literal(); } 1601 FunctionLiteral* FullCodeGenerator::literal() const { return info_->literal(); }
1607 1602
1608 #undef __ 1603 #undef __
1609 1604
1610 1605
1611 } // namespace internal 1606 } // namespace internal
1612 } // namespace v8 1607 } // namespace v8
OLDNEW
« no previous file with comments | « src/debug/debug.cc ('k') | src/ia32/macro-assembler-ia32.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698