s390: [interpreter] Make the binary op with Smi bytecode handlers collect type feedback.

src/s390/interface-descriptors-s390.cc

 // Copyright 2015 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #if V8_TARGET_ARCH_S390
 
 #include "src/interface-descriptors.h"
 
 namespace v8 {
 namespace internal {
@@ skipping 248 matching lines @@
   Register registers[] = {r3, r2};
   data->InitializePlatformSpecific(arraysize(registers), registers);
 }
 
 void BinaryOpWithAllocationSiteDescriptor::InitializePlatformSpecific(
     CallInterfaceDescriptorData* data) {
   Register registers[] = {r4, r3, r2};
   data->InitializePlatformSpecific(arraysize(registers), registers);
 }
 
+void BinaryOpWithVectorDescriptor::InitializePlatformSpecific(
+    CallInterfaceDescriptorData* data) {
+  // register state
+  // r3 -- lhs
+  // r2 -- rhs
+  // r6 -- slot id
+  // r5 -- vector
+  Register registers[] = {r3, r2, r6, r5};

[john.yan, 2016/08/23 16:31:55]

was it a intent to use r3 first?

[JaideepBajwa, 2016/08/23 17:46:34]

yes r3 should be first, the register array is passed as lhs then rhs

+  data->InitializePlatformSpecific(arraysize(registers), registers);
+}
+
 void CountOpDescriptor::InitializePlatformSpecific(
     CallInterfaceDescriptorData* data) {
   Register registers[] = {r4};
   data->InitializePlatformSpecific(arraysize(registers), registers);
 }
 
 void StringAddDescriptor::InitializePlatformSpecific(
     CallInterfaceDescriptorData* data) {
   Register registers[] = {r3, r2};
   data->InitializePlatformSpecific(arraysize(registers), registers);
@@ skipping 91 matching lines @@
       r3,  // the JSGeneratorObject to resume
       r4   // the resume mode (tagged)
   };
   data->InitializePlatformSpecific(arraysize(registers), registers);
 }
 
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_TARGET_ARCH_S390