Reviewers: Weiliang,

Message:
PTAL

Description:
X87: Eliminate redundant descriptor ElementTransitionAndStoreDescriptor.

port 26ffee2c718308bb85be072ed32479e4aa8e7e6b (r29789).

original commit message:

    It's just the same as StoreTransitionDescriptor.

BUG=

Please review this at https://codereview.chromium.org/1253573004/

Base URL: https://chromium.googlesource.com/v8/v8.git@master

Affected files (+2, -7 lines):
  M src/ic/x87/access-compiler-x87.cc
  M src/ic/x87/ic-compiler-x87.cc
  M src/x87/interface-descriptors-x87.cc


Index: src/ic/x87/access-compiler-x87.cc
diff --git a/src/ic/x87/access-compiler-x87.cc b/src/ic/x87/access-compiler-x87.cc index d5fde5d4b857e83e9f21989bef704aa324f55be3..fd2d784fdecab6041aa0912a7d41040e800fd07d 100644
--- a/src/ic/x87/access-compiler-x87.cc
+++ b/src/ic/x87/access-compiler-x87.cc
@@ -32,7 +32,7 @@ Register* PropertyAccessCompiler::store_calling_convention() {
   // receiver, name, scratch1, scratch2, scratch3.
   Register receiver = StoreDescriptor::ReceiverRegister();
   Register name = StoreDescriptor::NameRegister();
-  DCHECK(ebx.is(ElementTransitionAndStoreDescriptor::MapRegister()));
+  DCHECK(ebx.is(StoreTransitionDescriptor::MapRegister()));
   static Register registers[] = {receiver, name, ebx, edi, no_reg};
   return registers;
 }
Index: src/ic/x87/ic-compiler-x87.cc
diff --git a/src/ic/x87/ic-compiler-x87.cc b/src/ic/x87/ic-compiler-x87.cc
index 3697708037740dfad2400ace3da3d2ebe43c4b35..69ca69cc25f7aa393bf9dfaeffde089610385ed9 100644
--- a/src/ic/x87/ic-compiler-x87.cc
+++ b/src/ic/x87/ic-compiler-x87.cc
@@ -69,7 +69,7 @@ Handle<Code> PropertyICCompiler::CompilePolymorphic(MapHandleList* maps,
   // Polymorphic keyed stores may use the map register
   Register map_reg = scratch1();
   DCHECK(kind() != Code::KEYED_STORE_IC ||
-         map_reg.is(ElementTransitionAndStoreDescriptor::MapRegister()));
+         map_reg.is(StoreTransitionDescriptor::MapRegister()));
   __ mov(map_reg, FieldOperand(receiver(), HeapObject::kMapOffset));
   int receiver_count = maps->length();
   int number_of_handled_maps = 0;
Index: src/x87/interface-descriptors-x87.cc
diff --git a/src/x87/interface-descriptors-x87.cc b/src/x87/interface-descriptors-x87.cc index 279b42677c1e84cf0b09561935a05ff3508313cd..333564e09995afffc834146186bd7a14b4567cf8 100644
--- a/src/x87/interface-descriptors-x87.cc
+++ b/src/x87/interface-descriptors-x87.cc
@@ -46,11 +46,6 @@ const Register StoreGlobalViaContextDescriptor::NameRegister() { return ecx; } const Register StoreGlobalViaContextDescriptor::ValueRegister() { return eax; }


-const Register ElementTransitionAndStoreDescriptor::MapRegister() {
-  return ebx;
-}
-
-
 const Register InstanceofDescriptor::left() { return eax; }
 const Register InstanceofDescriptor::right() { return edx; }



--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to