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

Side by Side Diff: src/compiler/linkage.cc

Issue 595863002: Adding more missing deoptimization points in Turbofan. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Rebase Created 6 years, 2 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/js-inlining.cc ('k') | src/compiler/operator-properties-inl.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 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/linkage.h" 5 #include "src/compiler/linkage.h"
6 6
7 #include "src/code-stubs.h" 7 #include "src/code-stubs.h"
8 #include "src/compiler.h" 8 #include "src/compiler.h"
9 #include "src/compiler/node.h" 9 #include "src/compiler/node.h"
10 #include "src/compiler/pipeline.h" 10 #include "src/compiler/pipeline.h"
(...skipping 98 matching lines...) Expand 10 before | Expand all | Expand 10 after
109 109
110 110
111 // static 111 // static
112 bool Linkage::NeedsFrameState(Runtime::FunctionId function) { 112 bool Linkage::NeedsFrameState(Runtime::FunctionId function) {
113 if (!FLAG_turbo_deoptimization) { 113 if (!FLAG_turbo_deoptimization) {
114 return false; 114 return false;
115 } 115 }
116 // TODO(jarin) At the moment, we only add frame state for 116 // TODO(jarin) At the moment, we only add frame state for
117 // few chosen runtime functions. 117 // few chosen runtime functions.
118 switch (function) { 118 switch (function) {
119 case Runtime::kApply:
120 case Runtime::kArrayBufferNeuter:
121 case Runtime::kArrayConcat:
122 case Runtime::kBasicJSONStringify:
123 case Runtime::kCheckExecutionState:
124 case Runtime::kCollectStackTrace:
125 case Runtime::kCompileLazy:
126 case Runtime::kCompileOptimized:
127 case Runtime::kCompileString:
119 case Runtime::kDebugBreak: 128 case Runtime::kDebugBreak:
129 case Runtime::kDataViewSetInt8:
130 case Runtime::kDataViewSetUint8:
131 case Runtime::kDataViewSetInt16:
132 case Runtime::kDataViewSetUint16:
133 case Runtime::kDataViewSetInt32:
134 case Runtime::kDataViewSetUint32:
135 case Runtime::kDataViewSetFloat32:
136 case Runtime::kDataViewSetFloat64:
137 case Runtime::kDataViewGetInt8:
138 case Runtime::kDataViewGetUint8:
139 case Runtime::kDataViewGetInt16:
140 case Runtime::kDataViewGetUint16:
141 case Runtime::kDataViewGetInt32:
142 case Runtime::kDataViewGetUint32:
143 case Runtime::kDataViewGetFloat32:
144 case Runtime::kDataViewGetFloat64:
145 case Runtime::kDebugEvaluate:
120 case Runtime::kDebugGetLoadedScripts: 146 case Runtime::kDebugGetLoadedScripts:
147 case Runtime::kDebugGetPropertyDetails:
148 case Runtime::kDebugPromiseRejectEvent:
149 case Runtime::kDebugPromiseEvent:
150 case Runtime::kDeleteProperty:
121 case Runtime::kDeoptimizeFunction: 151 case Runtime::kDeoptimizeFunction:
152 case Runtime::kFunctionBindArguments:
153 case Runtime::kGetFrameCount:
154 case Runtime::kGetOwnProperty:
122 case Runtime::kInlineCallFunction: 155 case Runtime::kInlineCallFunction:
156 case Runtime::kInlineDateField:
157 case Runtime::kInlineRegExpExec:
158 case Runtime::kLiveEditGatherCompileInfo:
159 case Runtime::kLoadLookupSlot:
160 case Runtime::kLoadLookupSlotNoReferenceError:
161 case Runtime::kMaterializeRegExpLiteral:
162 case Runtime::kNewObjectFromBound:
163 case Runtime::kObjectFreeze:
164 case Runtime::kParseJson:
123 case Runtime::kPrepareStep: 165 case Runtime::kPrepareStep:
166 case Runtime::kPreventExtensions:
167 case Runtime::kRegExpCompile:
168 case Runtime::kRegExpExecMultiple:
169 case Runtime::kResolvePossiblyDirectEval:
170 // case Runtime::kSetPrototype:
124 case Runtime::kSetScriptBreakPoint: 171 case Runtime::kSetScriptBreakPoint:
125 case Runtime::kStackGuard: 172 case Runtime::kStackGuard:
126 case Runtime::kCheckExecutionState: 173 case Runtime::kStoreLookupSlot:
127 case Runtime::kDebugEvaluate: 174 case Runtime::kStringBuilderConcat:
128 case Runtime::kCollectStackTrace: 175 case Runtime::kStringReplaceGlobalRegExpWithString:
176 case Runtime::kThrowReferenceError:
177 case Runtime::kThrow:
178 case Runtime::kTypedArraySetFastCases:
179 case Runtime::kTypedArrayInitializeFromArrayLike:
129 return true; 180 return true;
130 default: 181 default:
131 return false; 182 return false;
132 } 183 }
133 } 184 }
134 185
135 186
136 //============================================================================== 187 //==============================================================================
137 // Provide unimplemented methods on unsupported architectures, to at least link. 188 // Provide unimplemented methods on unsupported architectures, to at least link.
138 //============================================================================== 189 //==============================================================================
(...skipping 22 matching lines...) Expand all
161 212
162 CallDescriptor* Linkage::GetSimplifiedCDescriptor(Zone* zone, 213 CallDescriptor* Linkage::GetSimplifiedCDescriptor(Zone* zone,
163 MachineSignature* sig) { 214 MachineSignature* sig) {
164 UNIMPLEMENTED(); 215 UNIMPLEMENTED();
165 return NULL; 216 return NULL;
166 } 217 }
167 #endif // !V8_TURBOFAN_BACKEND 218 #endif // !V8_TURBOFAN_BACKEND
168 } 219 }
169 } 220 }
170 } // namespace v8::internal::compiler 221 } // namespace v8::internal::compiler
OLDNEW
« no previous file with comments | « src/compiler/js-inlining.cc ('k') | src/compiler/operator-properties-inl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698