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

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

Issue 1314163003: [turbofan] Ensure stackcheck flags do something. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 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 | « no previous file | src/compiler/pipeline.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 #include "src/compiler/ast-graph-builder.h" 5 #include "src/compiler/ast-graph-builder.h"
6 6
7 #include "src/compiler.h" 7 #include "src/compiler.h"
8 #include "src/compiler/ast-loop-assignment-analyzer.h" 8 #include "src/compiler/ast-loop-assignment-analyzer.h"
9 #include "src/compiler/control-builders.h" 9 #include "src/compiler/control-builders.h"
10 #include "src/compiler/js-type-feedback.h" 10 #include "src/compiler/js-type-feedback.h"
(...skipping 2914 matching lines...) Expand 10 before | Expand all | Expand 10 after
2925 void AstGraphBuilder::VisitInScope(Statement* stmt, Scope* s, Node* context) { 2925 void AstGraphBuilder::VisitInScope(Statement* stmt, Scope* s, Node* context) {
2926 ContextScope scope(this, s, context); 2926 ContextScope scope(this, s, context);
2927 DCHECK(s->declarations()->is_empty()); 2927 DCHECK(s->declarations()->is_empty());
2928 Visit(stmt); 2928 Visit(stmt);
2929 } 2929 }
2930 2930
2931 2931
2932 void AstGraphBuilder::VisitIterationBody(IterationStatement* stmt, 2932 void AstGraphBuilder::VisitIterationBody(IterationStatement* stmt,
2933 LoopBuilder* loop) { 2933 LoopBuilder* loop) {
2934 ControlScopeForIteration scope(this, stmt, loop); 2934 ControlScopeForIteration scope(this, stmt, loop);
2935 if (FLAG_turbo_loop_stackcheck) { 2935 if (FLAG_turbo_loop_stackcheck || !info()->shared_info()->asm_function()) {
2936 Node* node = NewNode(javascript()->StackCheck()); 2936 Node* node = NewNode(javascript()->StackCheck());
2937 PrepareFrameState(node, stmt->StackCheckId()); 2937 PrepareFrameState(node, stmt->StackCheckId());
2938 } 2938 }
2939 Visit(stmt->body()); 2939 Visit(stmt->body());
2940 } 2940 }
2941 2941
2942 2942
2943 void AstGraphBuilder::VisitDelete(UnaryOperation* expr) { 2943 void AstGraphBuilder::VisitDelete(UnaryOperation* expr) {
2944 Node* value; 2944 Node* value;
2945 if (expr->expression()->IsVariableProxy()) { 2945 if (expr->expression()->IsVariableProxy()) {
(...skipping 1355 matching lines...) Expand 10 before | Expand all | Expand 10 after
4301 // Phi does not exist yet, introduce one. 4301 // Phi does not exist yet, introduce one.
4302 value = NewPhi(inputs, value, control); 4302 value = NewPhi(inputs, value, control);
4303 value->ReplaceInput(inputs - 1, other); 4303 value->ReplaceInput(inputs - 1, other);
4304 } 4304 }
4305 return value; 4305 return value;
4306 } 4306 }
4307 4307
4308 } // namespace compiler 4308 } // namespace compiler
4309 } // namespace internal 4309 } // namespace internal
4310 } // namespace v8 4310 } // namespace v8
OLDNEW
« no previous file with comments | « no previous file | src/compiler/pipeline.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698