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

Side by Side Diff: src/crankshaft/ia32/lithium-ia32.cc

Issue 1728423002: [crankshaft] Remove useless HCallJSFunction instruction. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@remove-dynamic-frame-alignment
Patch Set: Created 4 years, 9 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/crankshaft/ia32/lithium-ia32.h ('k') | src/crankshaft/mips/lithium-codegen-mips.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 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 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/crankshaft/ia32/lithium-ia32.h" 5 #include "src/crankshaft/ia32/lithium-ia32.h"
6 6
7 #include <sstream> 7 #include <sstream>
8 8
9 #if V8_TARGET_ARCH_IA32 9 #if V8_TARGET_ARCH_IA32
10 10
(...skipping 249 matching lines...) Expand 10 before | Expand all | Expand 10 after
260 260
261 261
262 void LInnerAllocatedObject::PrintDataTo(StringStream* stream) { 262 void LInnerAllocatedObject::PrintDataTo(StringStream* stream) {
263 stream->Add(" = "); 263 stream->Add(" = ");
264 base_object()->PrintTo(stream); 264 base_object()->PrintTo(stream);
265 stream->Add(" + "); 265 stream->Add(" + ");
266 offset()->PrintTo(stream); 266 offset()->PrintTo(stream);
267 } 267 }
268 268
269 269
270 void LCallJSFunction::PrintDataTo(StringStream* stream) {
271 stream->Add("= ");
272 function()->PrintTo(stream);
273 stream->Add("#%d / ", arity());
274 }
275
276
277 void LCallWithDescriptor::PrintDataTo(StringStream* stream) { 270 void LCallWithDescriptor::PrintDataTo(StringStream* stream) {
278 for (int i = 0; i < InputCount(); i++) { 271 for (int i = 0; i < InputCount(); i++) {
279 InputAt(i)->PrintTo(stream); 272 InputAt(i)->PrintTo(stream);
280 stream->Add(" "); 273 stream->Add(" ");
281 } 274 }
282 stream->Add("#%d / ", arity()); 275 stream->Add("#%d / ", arity());
283 } 276 }
284 277
285 278
286 void LLoadContextSlot::PrintDataTo(StringStream* stream) { 279 void LLoadContextSlot::PrintDataTo(StringStream* stream) {
(...skipping 785 matching lines...) Expand 10 before | Expand all | Expand 10 after
1072 return DefineAsRegister(new(zone()) LContext); 1065 return DefineAsRegister(new(zone()) LContext);
1073 } 1066 }
1074 1067
1075 1068
1076 LInstruction* LChunkBuilder::DoDeclareGlobals(HDeclareGlobals* instr) { 1069 LInstruction* LChunkBuilder::DoDeclareGlobals(HDeclareGlobals* instr) {
1077 LOperand* context = UseFixed(instr->context(), esi); 1070 LOperand* context = UseFixed(instr->context(), esi);
1078 return MarkAsCall(new(zone()) LDeclareGlobals(context), instr); 1071 return MarkAsCall(new(zone()) LDeclareGlobals(context), instr);
1079 } 1072 }
1080 1073
1081 1074
1082 LInstruction* LChunkBuilder::DoCallJSFunction(
1083 HCallJSFunction* instr) {
1084 LOperand* function = UseFixed(instr->function(), edi);
1085
1086 LCallJSFunction* result = new(zone()) LCallJSFunction(function);
1087
1088 return MarkAsCall(DefineFixed(result, eax), instr, CANNOT_DEOPTIMIZE_EAGERLY);
1089 }
1090
1091
1092 LInstruction* LChunkBuilder::DoCallWithDescriptor( 1075 LInstruction* LChunkBuilder::DoCallWithDescriptor(
1093 HCallWithDescriptor* instr) { 1076 HCallWithDescriptor* instr) {
1094 CallInterfaceDescriptor descriptor = instr->descriptor(); 1077 CallInterfaceDescriptor descriptor = instr->descriptor();
1095 LOperand* target = UseRegisterOrConstantAtStart(instr->target()); 1078 LOperand* target = UseRegisterOrConstantAtStart(instr->target());
1096 ZoneList<LOperand*> ops(instr->OperandCount(), zone()); 1079 ZoneList<LOperand*> ops(instr->OperandCount(), zone());
1097 // Target 1080 // Target
1098 ops.Add(target, zone()); 1081 ops.Add(target, zone());
1099 // Context 1082 // Context
1100 LOperand* op = UseFixed(instr->OperandAt(1), esi); 1083 LOperand* op = UseFixed(instr->OperandAt(1), esi);
1101 ops.Add(op, zone()); 1084 ops.Add(op, zone());
(...skipping 1520 matching lines...) Expand 10 before | Expand all | Expand 10 after
2622 LInstruction* LChunkBuilder::DoStoreFrameContext(HStoreFrameContext* instr) { 2605 LInstruction* LChunkBuilder::DoStoreFrameContext(HStoreFrameContext* instr) {
2623 LOperand* context = UseRegisterAtStart(instr->context()); 2606 LOperand* context = UseRegisterAtStart(instr->context());
2624 return new(zone()) LStoreFrameContext(context); 2607 return new(zone()) LStoreFrameContext(context);
2625 } 2608 }
2626 2609
2627 2610
2628 } // namespace internal 2611 } // namespace internal
2629 } // namespace v8 2612 } // namespace v8
2630 2613
2631 #endif // V8_TARGET_ARCH_IA32 2614 #endif // V8_TARGET_ARCH_IA32
OLDNEW
« no previous file with comments | « src/crankshaft/ia32/lithium-ia32.h ('k') | src/crankshaft/mips/lithium-codegen-mips.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698