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

Side by Side Diff: src/compiler/operator-properties-inl.h

Issue 517323002: Make FrameStates recursive (to be used for inlining). (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: jarin's comments. Created 6 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 | Annotate | Revision Log
« no previous file with comments | « src/compiler/node.h ('k') | src/compiler/raw-machine-assembler.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 2013 the V8 project authors. All rights reserved. 1 // Copyright 2013 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_OPERATOR_PROPERTIES_INL_H_ 5 #ifndef V8_COMPILER_OPERATOR_PROPERTIES_INL_H_
6 #define V8_COMPILER_OPERATOR_PROPERTIES_INL_H_ 6 #define V8_COMPILER_OPERATOR_PROPERTIES_INL_H_
7 7
8 #include "src/compiler/common-operator.h" 8 #include "src/compiler/common-operator.h"
9 #include "src/compiler/js-operator.h" 9 #include "src/compiler/js-operator.h"
10 #include "src/compiler/opcodes.h" 10 #include "src/compiler/opcodes.h"
(...skipping 22 matching lines...) Expand all
33 33
34 inline bool OperatorProperties::HasFrameStateInput(Operator* op) { 34 inline bool OperatorProperties::HasFrameStateInput(Operator* op) {
35 if (!FLAG_turbo_deoptimization) { 35 if (!FLAG_turbo_deoptimization) {
36 return false; 36 return false;
37 } 37 }
38 38
39 switch (op->opcode()) { 39 switch (op->opcode()) {
40 case IrOpcode::kJSCallFunction: 40 case IrOpcode::kJSCallFunction:
41 case IrOpcode::kJSCallConstruct: 41 case IrOpcode::kJSCallConstruct:
42 return true; 42 return true;
43 case IrOpcode::kFrameState:
44 return true;
43 case IrOpcode::kJSCallRuntime: { 45 case IrOpcode::kJSCallRuntime: {
44 Runtime::FunctionId function = 46 Runtime::FunctionId function =
45 reinterpret_cast<Operator1<Runtime::FunctionId>*>(op)->parameter(); 47 reinterpret_cast<Operator1<Runtime::FunctionId>*>(op)->parameter();
46 // TODO(jarin) At the moment, we only add frame state for 48 // TODO(jarin) At the moment, we only add frame state for
47 // few chosen runtime functions. 49 // few chosen runtime functions.
48 switch (function) { 50 switch (function) {
49 case Runtime::kDebugBreak: 51 case Runtime::kDebugBreak:
50 case Runtime::kDeoptimizeFunction: 52 case Runtime::kDeoptimizeFunction:
51 case Runtime::kSetScriptBreakPoint: 53 case Runtime::kSetScriptBreakPoint:
52 case Runtime::kDebugGetLoadedScripts: 54 case Runtime::kDebugGetLoadedScripts:
(...skipping 117 matching lines...) Expand 10 before | Expand all | Expand 10 after
170 opcode == IrOpcode::kDead || opcode == IrOpcode::kLoop || 172 opcode == IrOpcode::kDead || opcode == IrOpcode::kLoop ||
171 opcode == IrOpcode::kMerge || opcode == IrOpcode::kIfTrue || 173 opcode == IrOpcode::kMerge || opcode == IrOpcode::kIfTrue ||
172 opcode == IrOpcode::kIfFalse; 174 opcode == IrOpcode::kIfFalse;
173 } 175 }
174 176
175 } 177 }
176 } 178 }
177 } // namespace v8::internal::compiler 179 } // namespace v8::internal::compiler
178 180
179 #endif // V8_COMPILER_OPERATOR_PROPERTIES_INL_H_ 181 #endif // V8_COMPILER_OPERATOR_PROPERTIES_INL_H_
OLDNEW
« no previous file with comments | « src/compiler/node.h ('k') | src/compiler/raw-machine-assembler.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698