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

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

Issue 2333923008: [turbofan] Ensure AstGraphBuilder::CheckOsrEntry hits once. (Closed)
Patch Set: Created 4 years, 3 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/compilation-info.cc ('k') | no next file » | 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 #include "src/compiler/ast-graph-builder.h" 5 #include "src/compiler/ast-graph-builder.h"
6 6
7 #include "src/ast/compile-time-value.h" 7 #include "src/ast/compile-time-value.h"
8 #include "src/ast/scopes.h" 8 #include "src/ast/scopes.h"
9 #include "src/compilation-info.h" 9 #include "src/compilation-info.h"
10 #include "src/compiler.h" 10 #include "src/compiler.h"
(...skipping 3970 matching lines...) Expand 10 before | Expand all | Expand 10 after
3981 return input; 3981 return input;
3982 default: 3982 default:
3983 break; 3983 break;
3984 } 3984 }
3985 return nullptr; 3985 return nullptr;
3986 } 3986 }
3987 3987
3988 3988
3989 bool AstGraphBuilder::CheckOsrEntry(IterationStatement* stmt) { 3989 bool AstGraphBuilder::CheckOsrEntry(IterationStatement* stmt) {
3990 if (info()->osr_ast_id() == stmt->OsrEntryId()) { 3990 if (info()->osr_ast_id() == stmt->OsrEntryId()) {
3991 info()->set_osr_expr_stack_height(std::max( 3991 DCHECK_EQ(-1, info()->osr_expr_stack_height());
3992 environment()->stack_height(), info()->osr_expr_stack_height())); 3992 info()->set_osr_expr_stack_height(environment()->stack_height());
3993 return true; 3993 return true;
3994 } 3994 }
3995 return false; 3995 return false;
3996 } 3996 }
3997 3997
3998 3998
3999 void AstGraphBuilder::PrepareFrameState(Node* node, BailoutId ast_id, 3999 void AstGraphBuilder::PrepareFrameState(Node* node, BailoutId ast_id,
4000 OutputFrameStateCombine combine) { 4000 OutputFrameStateCombine combine) {
4001 if (OperatorProperties::HasFrameStateInput(node->op())) { 4001 if (OperatorProperties::HasFrameStateInput(node->op())) {
4002 DCHECK(ast_id.IsNone() || info()->shared_info()->VerifyBailoutId(ast_id)); 4002 DCHECK(ast_id.IsNone() || info()->shared_info()->VerifyBailoutId(ast_id));
(...skipping 321 matching lines...) Expand 10 before | Expand all | Expand 10 after
4324 // Phi does not exist yet, introduce one. 4324 // Phi does not exist yet, introduce one.
4325 value = NewPhi(inputs, value, control); 4325 value = NewPhi(inputs, value, control);
4326 value->ReplaceInput(inputs - 1, other); 4326 value->ReplaceInput(inputs - 1, other);
4327 } 4327 }
4328 return value; 4328 return value;
4329 } 4329 }
4330 4330
4331 } // namespace compiler 4331 } // namespace compiler
4332 } // namespace internal 4332 } // namespace internal
4333 } // namespace v8 4333 } // namespace v8
OLDNEW
« no previous file with comments | « src/compilation-info.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698