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

Side by Side Diff: src/compiler/ast-graph-builder.h

Issue 881303002: [turbofan] Add missing deopt for the assignment in the for-in statement. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Add test. Created 5 years, 10 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/ast.h ('k') | src/compiler/ast-graph-builder.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 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 #ifndef V8_COMPILER_AST_GRAPH_BUILDER_H_ 5 #ifndef V8_COMPILER_AST_GRAPH_BUILDER_H_
6 #define V8_COMPILER_AST_GRAPH_BUILDER_H_ 6 #define V8_COMPILER_AST_GRAPH_BUILDER_H_
7 7
8 #include "src/v8.h" 8 #include "src/v8.h"
9 9
10 #include "src/ast.h" 10 #include "src/ast.h"
(...skipping 179 matching lines...) Expand 10 before | Expand all | Expand 10 after
190 void VisitVoid(UnaryOperation* expr); 190 void VisitVoid(UnaryOperation* expr);
191 void VisitTypeof(UnaryOperation* expr); 191 void VisitTypeof(UnaryOperation* expr);
192 void VisitNot(UnaryOperation* expr); 192 void VisitNot(UnaryOperation* expr);
193 193
194 // Dispatched from VisitBinaryOperation. 194 // Dispatched from VisitBinaryOperation.
195 void VisitComma(BinaryOperation* expr); 195 void VisitComma(BinaryOperation* expr);
196 void VisitLogicalExpression(BinaryOperation* expr); 196 void VisitLogicalExpression(BinaryOperation* expr);
197 void VisitArithmeticExpression(BinaryOperation* expr); 197 void VisitArithmeticExpression(BinaryOperation* expr);
198 198
199 // Dispatched from VisitForInStatement. 199 // Dispatched from VisitForInStatement.
200 void VisitForInAssignment(Expression* expr, Node* value); 200 void VisitForInAssignment(Expression* expr, Node* value,
201 BailoutId bailout_id);
201 202
202 // Dispatched from VisitClassLiteral. 203 // Dispatched from VisitClassLiteral.
203 void VisitClassLiteralContents(ClassLiteral* expr); 204 void VisitClassLiteralContents(ClassLiteral* expr);
204 205
205 // Builds deoptimization for a given node. 206 // Builds deoptimization for a given node.
206 void PrepareFrameState( 207 void PrepareFrameState(
207 Node* node, BailoutId ast_id, 208 Node* node, BailoutId ast_id,
208 OutputFrameStateCombine combine = OutputFrameStateCombine::Ignore()); 209 OutputFrameStateCombine combine = OutputFrameStateCombine::Ignore());
209 210
210 BitVector* GetVariablesAssignedInLoop(IterationStatement* stmt); 211 BitVector* GetVariablesAssignedInLoop(IterationStatement* stmt);
(...skipping 243 matching lines...) Expand 10 before | Expand all | Expand 10 after
454 455
455 Scope* AstGraphBuilder::current_scope() const { 456 Scope* AstGraphBuilder::current_scope() const {
456 return execution_context_->scope(); 457 return execution_context_->scope();
457 } 458 }
458 459
459 } // namespace compiler 460 } // namespace compiler
460 } // namespace internal 461 } // namespace internal
461 } // namespace v8 462 } // namespace v8
462 463
463 #endif // V8_COMPILER_AST_GRAPH_BUILDER_H_ 464 #endif // V8_COMPILER_AST_GRAPH_BUILDER_H_
OLDNEW
« no previous file with comments | « src/ast.h ('k') | src/compiler/ast-graph-builder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698