src/compiler/linkage.cc - Issue 1957393004: Allow Turbofan optimization of Ignition generators

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

Issue 1957393004: Allow Turbofan optimization of Ignition generators (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master

Patch Set: Created 4 years, 7 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

OLD	NEW
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/ast/scopes.h"	5 #include "src/ast/scopes.h"

6 #include "src/code-stubs.h"	6 #include "src/code-stubs.h"

7 #include "src/compiler.h"	7 #include "src/compiler.h"

8 #include "src/compiler/common-operator.h"	8 #include "src/compiler/common-operator.h"

9 #include "src/compiler/frame.h"	9 #include "src/compiler/frame.h"

10 #include "src/compiler/linkage.h"	10 #include "src/compiler/linkage.h"

(...skipping 118 matching lines...) Expand 10 before \| Expand all \| Expand 10 after Loading...
129 return nullptr; // TODO(titzer): ?	129 return nullptr; // TODO(titzer): ?

130 }	130 }

131	131

132	132

133 // static	133 // static

134 int Linkage::FrameStateInputCount(Runtime::FunctionId function) {	134 int Linkage::FrameStateInputCount(Runtime::FunctionId function) {

135 // Most runtime functions need a FrameState. A few chosen ones that we know	135 // Most runtime functions need a FrameState. A few chosen ones that we know

136 // not to call into arbitrary JavaScript, not to throw, and not to deoptimize	136 // not to call into arbitrary JavaScript, not to throw, and not to deoptimize

137 // are blacklisted here and can be called without a FrameState.	137 // are blacklisted here and can be called without a FrameState.

138 switch (function) {	138 switch (function) {

	139 case Runtime::kAbort:

139 case Runtime::kAllocateInTargetSpace:	140 case Runtime::kAllocateInTargetSpace:

140 case Runtime::kCreateIterResultObject:	141 case Runtime::kCreateIterResultObject:

141 case Runtime::kDefineDataPropertyInLiteral:	142 case Runtime::kDefineDataPropertyInLiteral:

142 case Runtime::kDefineGetterPropertyUnchecked: // TODO(jarin): Is it safe?	143 case Runtime::kDefineGetterPropertyUnchecked: // TODO(jarin): Is it safe?

143 case Runtime::kDefineSetterPropertyUnchecked: // TODO(jarin): Is it safe?	144 case Runtime::kDefineSetterPropertyUnchecked: // TODO(jarin): Is it safe?

144 case Runtime::kFinalizeClassDefinition: // TODO(conradw): Is it safe?	145 case Runtime::kFinalizeClassDefinition: // TODO(conradw): Is it safe?

145 case Runtime::kForInDone:	146 case Runtime::kForInDone:

146 case Runtime::kForInStep:	147 case Runtime::kForInStep:

	148 case Runtime::kGeneratorSetContext:

	149 case Runtime::kGeneratorGetContinuation:

	150 case Runtime::kGeneratorSetContinuation:

	151 case Runtime::kGeneratorReadRegister:

	152 case Runtime::kGeneratorWriteRegister:

147 case Runtime::kGetSuperConstructor:	153 case Runtime::kGetSuperConstructor:

148 case Runtime::kIsFunction:	154 case Runtime::kIsFunction:

149 case Runtime::kNewClosure:	155 case Runtime::kNewClosure:

150 case Runtime::kNewClosure_Tenured:	156 case Runtime::kNewClosure_Tenured:

151 case Runtime::kNewFunctionContext:	157 case Runtime::kNewFunctionContext:

152 case Runtime::kPushBlockContext:	158 case Runtime::kPushBlockContext:

153 case Runtime::kPushCatchContext:	159 case Runtime::kPushCatchContext:

154 case Runtime::kReThrow:	160 case Runtime::kReThrow:

155 case Runtime::kStringCompare:	161 case Runtime::kStringCompare:

156 case Runtime::kStringEqual:	162 case Runtime::kStringEqual:

(...skipping 360 matching lines...) Expand 10 before \| Expand all \| Expand 10 after Loading...
517 } else {	523 } else {

518 DCHECK(loc == regloc(kContextRegister));	524 DCHECK(loc == regloc(kContextRegister));

519 return LinkageLocation::ForCalleeFrameSlot(Frame::kContextSlot);	525 return LinkageLocation::ForCalleeFrameSlot(Frame::kContextSlot);

520 }	526 }

521 }	527 }

522	528

523	529

524 } // namespace compiler	530 } // namespace compiler

525 } // namespace internal	531 } // namespace internal

526 } // namespace v8	532 } // namespace v8

OLD	NEW

« src/compiler/bytecode-graph-builder.cc ('K') | « src/compiler/js-graph.cc ('k') | src/compiler/register-allocator.cc » ('j') | src/runtime/runtime-generator.cc » ('J')