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

Side by Side Diff: src/crankshaft/x64/lithium-x64.h

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/x64/lithium-codegen-x64.cc ('k') | src/crankshaft/x64/lithium-x64.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 #ifndef V8_CRANKSHAFT_X64_LITHIUM_X64_H_ 5 #ifndef V8_CRANKSHAFT_X64_LITHIUM_X64_H_
6 #define V8_CRANKSHAFT_X64_LITHIUM_X64_H_ 6 #define V8_CRANKSHAFT_X64_LITHIUM_X64_H_
7 7
8 #include "src/crankshaft/hydrogen.h" 8 #include "src/crankshaft/hydrogen.h"
9 #include "src/crankshaft/lithium.h" 9 #include "src/crankshaft/lithium.h"
10 #include "src/crankshaft/lithium-allocator.h" 10 #include "src/crankshaft/lithium-allocator.h"
(...skipping 11 matching lines...) Expand all
22 V(AddI) \ 22 V(AddI) \
23 V(Allocate) \ 23 V(Allocate) \
24 V(ApplyArguments) \ 24 V(ApplyArguments) \
25 V(ArgumentsElements) \ 25 V(ArgumentsElements) \
26 V(ArgumentsLength) \ 26 V(ArgumentsLength) \
27 V(ArithmeticD) \ 27 V(ArithmeticD) \
28 V(ArithmeticT) \ 28 V(ArithmeticT) \
29 V(BitI) \ 29 V(BitI) \
30 V(BoundsCheck) \ 30 V(BoundsCheck) \
31 V(Branch) \ 31 V(Branch) \
32 V(CallJSFunction) \
33 V(CallWithDescriptor) \ 32 V(CallWithDescriptor) \
34 V(CallNewArray) \ 33 V(CallNewArray) \
35 V(CallRuntime) \ 34 V(CallRuntime) \
36 V(CheckArrayBufferNotNeutered) \ 35 V(CheckArrayBufferNotNeutered) \
37 V(CheckInstanceType) \ 36 V(CheckInstanceType) \
38 V(CheckMaps) \ 37 V(CheckMaps) \
39 V(CheckMapValue) \ 38 V(CheckMapValue) \
40 V(CheckNonSmi) \ 39 V(CheckNonSmi) \
41 V(CheckSmi) \ 40 V(CheckSmi) \
42 V(CheckValue) \ 41 V(CheckValue) \
(...skipping 1664 matching lines...) Expand 10 before | Expand all | Expand 10 after
1707 inputs_[0] = context; 1706 inputs_[0] = context;
1708 } 1707 }
1709 1708
1710 LOperand* context() { return inputs_[0]; } 1709 LOperand* context() { return inputs_[0]; }
1711 1710
1712 DECLARE_CONCRETE_INSTRUCTION(DeclareGlobals, "declare-globals") 1711 DECLARE_CONCRETE_INSTRUCTION(DeclareGlobals, "declare-globals")
1713 DECLARE_HYDROGEN_ACCESSOR(DeclareGlobals) 1712 DECLARE_HYDROGEN_ACCESSOR(DeclareGlobals)
1714 }; 1713 };
1715 1714
1716 1715
1717 class LCallJSFunction final : public LTemplateInstruction<1, 1, 0> {
1718 public:
1719 explicit LCallJSFunction(LOperand* function) {
1720 inputs_[0] = function;
1721 }
1722
1723 LOperand* function() { return inputs_[0]; }
1724
1725 DECLARE_CONCRETE_INSTRUCTION(CallJSFunction, "call-js-function")
1726 DECLARE_HYDROGEN_ACCESSOR(CallJSFunction)
1727
1728 void PrintDataTo(StringStream* stream) override;
1729
1730 int arity() const { return hydrogen()->argument_count() - 1; }
1731 };
1732
1733
1734 class LCallWithDescriptor final : public LTemplateResultInstruction<1> { 1716 class LCallWithDescriptor final : public LTemplateResultInstruction<1> {
1735 public: 1717 public:
1736 LCallWithDescriptor(CallInterfaceDescriptor descriptor, 1718 LCallWithDescriptor(CallInterfaceDescriptor descriptor,
1737 const ZoneList<LOperand*>& operands, Zone* zone) 1719 const ZoneList<LOperand*>& operands, Zone* zone)
1738 : inputs_(descriptor.GetRegisterParameterCount() + 1720 : inputs_(descriptor.GetRegisterParameterCount() +
1739 kImplicitRegisterParameterCount, 1721 kImplicitRegisterParameterCount,
1740 zone) { 1722 zone) {
1741 DCHECK(descriptor.GetRegisterParameterCount() + 1723 DCHECK(descriptor.GetRegisterParameterCount() +
1742 kImplicitRegisterParameterCount == 1724 kImplicitRegisterParameterCount ==
1743 operands.length()); 1725 operands.length());
(...skipping 927 matching lines...) Expand 10 before | Expand all | Expand 10 after
2671 DISALLOW_COPY_AND_ASSIGN(LChunkBuilder); 2653 DISALLOW_COPY_AND_ASSIGN(LChunkBuilder);
2672 }; 2654 };
2673 2655
2674 #undef DECLARE_HYDROGEN_ACCESSOR 2656 #undef DECLARE_HYDROGEN_ACCESSOR
2675 #undef DECLARE_CONCRETE_INSTRUCTION 2657 #undef DECLARE_CONCRETE_INSTRUCTION
2676 2658
2677 } // namespace internal 2659 } // namespace internal
2678 } // namespace v8 2660 } // namespace v8
2679 2661
2680 #endif // V8_CRANKSHAFT_X64_LITHIUM_X64_H_ 2662 #endif // V8_CRANKSHAFT_X64_LITHIUM_X64_H_
OLDNEW
« no previous file with comments | « src/crankshaft/x64/lithium-codegen-x64.cc ('k') | src/crankshaft/x64/lithium-x64.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698