AuroraMiddleware/v8
1
0
Fork 0
Code Issues 2 Pull Requests Releases Wiki Activity

InterfaceDescriptor becomes CallInterfaceDescriptor.

Browse Source 
There was no difference between these two classes in a hierarchical
relationship.

R=yangguo@chromium.org

Review URL: https://codereview.chromium.org/532473002

git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@23560 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
This commit is contained in:
mvstanton@chromium.org 2014-09-01 12:15:25 +00:00
parent ae2b08f420
commit 3a7b5b44c1
15 changed files with 58 additions and 91 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/arm/interface-descriptors-arm.cc
View File

@ -11,7 +11,7 @@
namespace v8 {
namespace internal {
const Register InterfaceDescriptor::ContextRegister() { return cp; }
const Register CallInterfaceDescriptor::ContextRegister() { return cp; }
void CallDescriptors::InitializeForIsolate(Isolate* isolate) {

2
src/arm/lithium-arm.cc
View File

@ -1082,7 +1082,7 @@ LInstruction* LChunkBuilder::DoCallJSFunction(
LInstruction* LChunkBuilder::DoCallWithDescriptor(
HCallWithDescriptor* instr) {
const InterfaceDescriptor* descriptor = instr->descriptor();
const CallInterfaceDescriptor* descriptor = instr->descriptor();
LOperand* target = UseRegisterOrConstantAtStart(instr->target());
ZoneList<LOperand*> ops(instr->OperandCount(), zone());

9
src/arm/lithium-arm.h
View File

@ -1874,9 +1874,8 @@ class LCallJSFunction V8_FINAL : public LTemplateInstruction<1, 1, 0> {
class LCallWithDescriptor V8_FINAL : public LTemplateResultInstruction<1> {
public:
LCallWithDescriptor(const InterfaceDescriptor* descriptor,
const ZoneList<LOperand*>& operands,
Zone* zone)
LCallWithDescriptor(const CallInterfaceDescriptor* descriptor,
const ZoneList<LOperand*>& operands, Zone* zone)
: descriptor_(descriptor),
inputs_(descriptor->GetRegisterParameterCount() + 1, zone) {
DCHECK(descriptor->GetRegisterParameterCount() + 1 == operands.length());
@ -1885,7 +1884,7 @@ class LCallWithDescriptor V8_FINAL : public LTemplateResultInstruction<1> {
LOperand* target() const { return inputs_[0]; }
const InterfaceDescriptor* descriptor() { return descriptor_; }
const CallInterfaceDescriptor* descriptor() { return descriptor_; }
private:
DECLARE_CONCRETE_INSTRUCTION(CallWithDescriptor, "call-with-descriptor")
@ -1895,7 +1894,7 @@ class LCallWithDescriptor V8_FINAL : public LTemplateResultInstruction<1> {
int arity() const { return hydrogen()->argument_count() - 1; }
const InterfaceDescriptor* descriptor_;
const CallInterfaceDescriptor* descriptor_;
ZoneList<LOperand*> inputs_;
// Iterator support.

2
src/arm64/interface-descriptors-arm64.cc
View File

@ -11,7 +11,7 @@
namespace v8 {
namespace internal {
const Register InterfaceDescriptor::ContextRegister() { return cp; }
const Register CallInterfaceDescriptor::ContextRegister() { return cp; }
void CallDescriptors::InitializeForIsolate(Isolate* isolate) {

2
src/arm64/lithium-arm64.cc
View File

@ -1036,7 +1036,7 @@ LInstruction* LChunkBuilder::DoCallJSFunction(
LInstruction* LChunkBuilder::DoCallWithDescriptor(
HCallWithDescriptor* instr) {
const InterfaceDescriptor* descriptor = instr->descriptor();
const CallInterfaceDescriptor* descriptor = instr->descriptor();
LOperand* target = UseRegisterOrConstantAtStart(instr->target());
ZoneList<LOperand*> ops(instr->OperandCount(), zone());

9
src/arm64/lithium-arm64.h
View File

@ -1523,9 +1523,8 @@ class LInteger32ToDouble V8_FINAL : public LTemplateInstruction<1, 1, 0> {
class LCallWithDescriptor V8_FINAL : public LTemplateResultInstruction<1> {
public:
LCallWithDescriptor(const InterfaceDescriptor* descriptor,
const ZoneList<LOperand*>& operands,
Zone* zone)
LCallWithDescriptor(const CallInterfaceDescriptor* descriptor,
const ZoneList<LOperand*>& operands, Zone* zone)
: descriptor_(descriptor),
inputs_(descriptor->GetRegisterParameterCount() + 1, zone) {
DCHECK(descriptor->GetRegisterParameterCount() + 1 == operands.length());
@ -1534,7 +1533,7 @@ class LCallWithDescriptor V8_FINAL : public LTemplateResultInstruction<1> {
LOperand* target() const { return inputs_[0]; }
const InterfaceDescriptor* descriptor() { return descriptor_; }
const CallInterfaceDescriptor* descriptor() { return descriptor_; }
private:
DECLARE_CONCRETE_INSTRUCTION(CallWithDescriptor, "call-with-descriptor")
@ -1544,7 +1543,7 @@ class LCallWithDescriptor V8_FINAL : public LTemplateResultInstruction<1> {
int arity() const { return hydrogen()->argument_count() - 1; }
const InterfaceDescriptor* descriptor_;
const CallInterfaceDescriptor* descriptor_;
ZoneList<LOperand*> inputs_;
// Iterator support.

19
src/hydrogen-instructions.h
View File

@ -2318,10 +2318,9 @@ class HCallJSFunction V8_FINAL : public HCall<1> {
class HCallWithDescriptor V8_FINAL : public HInstruction {
public:
static HCallWithDescriptor* New(Zone* zone, HValue* context,
HValue* target,
static HCallWithDescriptor* New(Zone* zone, HValue* context, HValue* target,
int argument_count,
const InterfaceDescriptor* descriptor,
const CallInterfaceDescriptor* descriptor,
const Vector<HValue*>& operands) {
DCHECK(operands.length() == descriptor->GetEnvironmentLength());
HCallWithDescriptor* res =
@ -2362,9 +2361,7 @@ class HCallWithDescriptor V8_FINAL : public HInstruction {
return -argument_count_;
}
const InterfaceDescriptor* descriptor() const {
return descriptor_;
}
const CallInterfaceDescriptor* descriptor() const { return descriptor_; }
HValue* target() {
return OperandAt(0);
@ -2374,11 +2371,9 @@ class HCallWithDescriptor V8_FINAL : public HInstruction {
private:
// The argument count includes the receiver.
HCallWithDescriptor(HValue* target,
int argument_count,
const InterfaceDescriptor* descriptor,
const Vector<HValue*>& operands,
Zone* zone)
HCallWithDescriptor(HValue* target, int argument_count,
const CallInterfaceDescriptor* descriptor,
const Vector<HValue*>& operands, Zone* zone)
: descriptor_(descriptor),
values_(descriptor->GetEnvironmentLength() + 1, zone) {
argument_count_ = argument_count;
@ -2400,7 +2395,7 @@ class HCallWithDescriptor V8_FINAL : public HInstruction {
values_[index] = value;
}
const InterfaceDescriptor* descriptor_;
const CallInterfaceDescriptor* descriptor_;
ZoneList<HValue*> values_;
int argument_count_;
};

2
src/ia32/interface-descriptors-ia32.cc
View File

@ -12,7 +12,7 @@
namespace v8 {
namespace internal {
const Register InterfaceDescriptor::ContextRegister() { return esi; }
const Register CallInterfaceDescriptor::ContextRegister() { return esi; }
void CallDescriptors::InitializeForIsolate(Isolate* isolate) {

2
src/ia32/lithium-ia32.cc
View File

@ -1123,7 +1123,7 @@ LInstruction* LChunkBuilder::DoCallJSFunction(
LInstruction* LChunkBuilder::DoCallWithDescriptor(
HCallWithDescriptor* instr) {
const InterfaceDescriptor* descriptor = instr->descriptor();
const CallInterfaceDescriptor* descriptor = instr->descriptor();
LOperand* target = UseRegisterOrConstantAtStart(instr->target());
ZoneList<LOperand*> ops(instr->OperandCount(), zone());
ops.Add(target, zone());

5
src/ia32/lithium-ia32.h
View File

@ -1891,9 +1891,8 @@ class LCallJSFunction V8_FINAL : public LTemplateInstruction<1, 1, 0> {
class LCallWithDescriptor V8_FINAL : public LTemplateResultInstruction<1> {
public:
LCallWithDescriptor(const InterfaceDescriptor* descriptor,
const ZoneList<LOperand*>& operands,
Zone* zone)
LCallWithDescriptor(const CallInterfaceDescriptor* descriptor,
const ZoneList<LOperand*>& operands, Zone* zone)
: inputs_(descriptor->GetRegisterParameterCount() + 1, zone) {
DCHECK(descriptor->GetRegisterParameterCount() + 1 == operands.length());
inputs_.AddAll(operands, zone);

24
src/interface-descriptors.cc
View File

@ -10,10 +10,7 @@
namespace v8 {
namespace internal {
InterfaceDescriptor::InterfaceDescriptor() : register_param_count_(-1) {}
void InterfaceDescriptor::Initialize(
void CallInterfaceDescriptor::Initialize(
int register_parameter_count, Register* registers,
Representation* register_param_representations,
PlatformInterfaceDescriptor* platform_descriptor) {
@ -45,20 +42,11 @@ void InterfaceDescriptor::Initialize(
}
void CallInterfaceDescriptor::Initialize(
int register_parameter_count, Register* registers,
Representation* param_representations,
PlatformInterfaceDescriptor* platform_descriptor) {
InterfaceDescriptor::Initialize(register_parameter_count, registers,
param_representations, platform_descriptor);
}
void CallDescriptors::InitializeForIsolateAllPlatforms(Isolate* isolate) {
{
CallInterfaceDescriptor* descriptor =
isolate->call_descriptor(CallDescriptorKey::LoadICCall);
Register registers[] = {InterfaceDescriptor::ContextRegister(),
Register registers[] = {CallInterfaceDescriptor::ContextRegister(),
LoadConvention::ReceiverRegister(),
LoadConvention::NameRegister()};
descriptor->Initialize(arraysize(registers), registers, NULL);
@ -66,7 +54,7 @@ void CallDescriptors::InitializeForIsolateAllPlatforms(Isolate* isolate) {
{
CallInterfaceDescriptor* descriptor =
isolate->call_descriptor(CallDescriptorKey::StoreICCall);
Register registers[] = {InterfaceDescriptor::ContextRegister(),
Register registers[] = {CallInterfaceDescriptor::ContextRegister(),
StoreConvention::ReceiverRegister(),
StoreConvention::NameRegister(),
StoreConvention::ValueRegister()};
@ -76,7 +64,7 @@ void CallDescriptors::InitializeForIsolateAllPlatforms(Isolate* isolate) {
CallInterfaceDescriptor* descriptor = isolate->call_descriptor(
CallDescriptorKey::ElementTransitionAndStoreCall);
Register registers[] = {
InterfaceDescriptor::ContextRegister(),
CallInterfaceDescriptor::ContextRegister(),
StoreConvention::ValueRegister(), StoreConvention::MapRegister(),
StoreConvention::NameRegister(), StoreConvention::ReceiverRegister()};
descriptor->Initialize(arraysize(registers), registers, NULL);
@ -84,7 +72,7 @@ void CallDescriptors::InitializeForIsolateAllPlatforms(Isolate* isolate) {
{
CallInterfaceDescriptor* descriptor =
isolate->call_descriptor(CallDescriptorKey::InstanceofCall);
Register registers[] = {InterfaceDescriptor::ContextRegister(),
Register registers[] = {CallInterfaceDescriptor::ContextRegister(),
InstanceofConvention::left(),
InstanceofConvention::right()};
descriptor->Initialize(arraysize(registers), registers, NULL);
@ -92,7 +80,7 @@ void CallDescriptors::InitializeForIsolateAllPlatforms(Isolate* isolate) {
{
CallInterfaceDescriptor* descriptor =
isolate->call_descriptor(CallDescriptorKey::VectorLoadICCall);
Register registers[] = {InterfaceDescriptor::ContextRegister(),
Register registers[] = {CallInterfaceDescriptor::ContextRegister(),
FullVectorLoadConvention::ReceiverRegister(),
FullVectorLoadConvention::NameRegister(),
FullVectorLoadConvention::SlotRegister(),

42
src/interface-descriptors.h
View File

@ -13,8 +13,21 @@ namespace internal {
class PlatformInterfaceDescriptor;
class InterfaceDescriptor {
class CallInterfaceDescriptor {
public:
CallInterfaceDescriptor() : register_param_count_(-1) {}
// A copy of the passed in registers and param_representations is made
// and owned by the CallInterfaceDescriptor.
// TODO(mvstanton): Instead of taking parallel arrays register and
// param_representations, how about a struct that puts the representation
// and register side by side (eg, RegRep(r1, Representation::Tagged()).
// The same should go for the CodeStubInterfaceDescriptor class.
void Initialize(int register_parameter_count, Register* registers,
Representation* param_representations,
PlatformInterfaceDescriptor* platform_descriptor = NULL);
bool IsInitialized() const { return register_param_count_ >= 0; }
int GetEnvironmentLength() const { return register_param_count_; }
@ -55,14 +68,6 @@ class InterfaceDescriptor {
static const Register ContextRegister();
protected:
InterfaceDescriptor();
virtual ~InterfaceDescriptor() {}
void Initialize(int register_parameter_count, Register* registers,
Representation* register_param_representations,
PlatformInterfaceDescriptor* platform_descriptor = NULL);
private:
int register_param_count_;
@ -79,7 +84,7 @@ class InterfaceDescriptor {
PlatformInterfaceDescriptor* platform_specific_descriptor_;
DISALLOW_COPY_AND_ASSIGN(InterfaceDescriptor);
DISALLOW_COPY_AND_ASSIGN(CallInterfaceDescriptor);
};
@ -118,23 +123,6 @@ enum CallDescriptorKey {
};
class CallInterfaceDescriptor : public InterfaceDescriptor {
public:
CallInterfaceDescriptor() {}
// A copy of the passed in registers and param_representations is made
// and owned by the CallInterfaceDescriptor.
// TODO(mvstanton): Instead of taking parallel arrays register and
// param_representations, how about a struct that puts the representation
// and register side by side (eg, RegRep(r1, Representation::Tagged()).
// The same should go for the CodeStubInterfaceDescriptor class.
void Initialize(int register_parameter_count, Register* registers,
Representation* param_representations,
PlatformInterfaceDescriptor* platform_descriptor = NULL);
};
class CallDescriptors {
public:
static void InitializeForIsolate(Isolate* isolate);

2
src/x64/interface-descriptors-x64.cc
View File

@ -11,7 +11,7 @@
namespace v8 {
namespace internal {
const Register InterfaceDescriptor::ContextRegister() { return rsi; }
const Register CallInterfaceDescriptor::ContextRegister() { return rsi; }
void CallDescriptors::InitializeForIsolate(Isolate* isolate) {

2
src/x64/lithium-x64.cc
View File

@ -1102,7 +1102,7 @@ LInstruction* LChunkBuilder::DoCallJSFunction(
LInstruction* LChunkBuilder::DoCallWithDescriptor(
HCallWithDescriptor* instr) {
const InterfaceDescriptor* descriptor = instr->descriptor();
const CallInterfaceDescriptor* descriptor = instr->descriptor();
LOperand* target = UseRegisterOrConstantAtStart(instr->target());
ZoneList<LOperand*> ops(instr->OperandCount(), zone());

5
src/x64/lithium-x64.h
View File

@ -1877,9 +1877,8 @@ class LCallJSFunction V8_FINAL : public LTemplateInstruction<1, 1, 0> {
class LCallWithDescriptor V8_FINAL : public LTemplateResultInstruction<1> {
public:
LCallWithDescriptor(const InterfaceDescriptor* descriptor,
const ZoneList<LOperand*>& operands,
Zone* zone)
LCallWithDescriptor(const CallInterfaceDescriptor* descriptor,
const ZoneList<LOperand*>& operands, Zone* zone)
: inputs_(descriptor->GetRegisterParameterCount() + 1, zone) {
DCHECK(descriptor->GetRegisterParameterCount() + 1 == operands.length());
inputs_.AddAll(operands, zone);