Cleanup interface descriptors to reflect that vectors are part of stores.

Cleanup interface descriptors to reflect that vectors are part of stores.

All stores require a Slot parameter so VectorStoreICTrampolineDescriptor is a new StoreDescriptor.

VectorStoreICDescriptor renamed to StoreWithVectorDescriptor.

Committed: https://crrev.com/53be25303970574b7d7ee3e3cd417c615d21dac2
Cr-Commit-Position: refs/heads/master@{#37779}

[Igor Sheludko, 2016-07-13 19:41:20]

The CQ bit was checked by ishell@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-13 19:41:27]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Igor Sheludko, 2016-07-13 19:42:54]

Description was changed from

==========
Cleanup interface descriptors to reflect that vectors are part of stores.
==========

to

==========
Cleanup interface descriptors to reflect that vectors are part of stores.

StoreDescriptor always has a Slot parameter, so separate
VectorStoreICTrampolineDescriptor is no longer needed.

VectorStoreICDescriptor -> StoreWithVectorDescriptor
==========

[Igor Sheludko, 2016-07-13 19:44:25]

Description was changed from

==========
Cleanup interface descriptors to reflect that vectors are part of stores.

StoreDescriptor always has a Slot parameter, so separate
VectorStoreICTrampolineDescriptor is no longer needed.

VectorStoreICDescriptor -> StoreWithVectorDescriptor
==========

to

==========
Cleanup interface descriptors to reflect that vectors are part of stores.

All stores require a Slot parameter so VectorStoreICTrampolineDescriptor is a
new StoreDescriptor.

VectorStoreICDescriptor renamed to StoreWithVectorDescriptor.
==========

[Igor Sheludko, 2016-07-13 19:45:54]

ishell@chromium.org changed reviewers:
+ jkummerow@chromium.org

[Igor Sheludko, 2016-07-13 19:45:55]

PTAL

[commit-bot: I haz the power, 2016-07-13 19:57:16]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-13 19:57:17]

Dry run: Try jobs failed on following builders:
  v8_linux64_asan_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_asan_rel_ng/buil...)
  v8_linux64_asan_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_asan_rel_ng_trig...)

[Igor Sheludko, 2016-07-13 22:17:43]

The CQ bit was checked by ishell@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-13 22:17:51]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-13 22:41:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-13 22:41:02]

Dry run: This issue passed the CQ dry run.

[Jakob Kummerow, 2016-07-14 11:08:05]

lgtm

https://codereview.chromium.org/2147043002/diff/20001/src/interface-descripto...
File src/interface-descriptors.cc (right):

https://codereview.chromium.org/2147043002/diff/20001/src/interface-descripto...
src/interface-descriptors.cc:134: Isolate* isolate, int paramater_count) {
nit: s/mater/meter/. While you're at it, you can fix all 9 occurrences of that
typo in this file :-)

[Igor Sheludko, 2016-07-14 13:05:23]

https://codereview.chromium.org/2147043002/diff/20001/src/interface-descripto...
File src/interface-descriptors.cc (right):

https://codereview.chromium.org/2147043002/diff/20001/src/interface-descripto...
src/interface-descriptors.cc:134: Isolate* isolate, int paramater_count) {
On 2016/07/14 11:08:05, Jakob wrote:
> nit: s/mater/meter/. While you're at it, you can fix all 9 occurrences of that
> typo in this file :-)

Done.

[Igor Sheludko, 2016-07-14 13:05:59]

The CQ bit was checked by ishell@chromium.org

[Igor Sheludko, 2016-07-14 13:05:59]

The patchset sent to the CQ was uploaded after l-g-t-m from
jkummerow@chromium.org
Link to the patchset: https://codereview.chromium.org/2147043002/#ps40001
(title: "Addressing comments")

[commit-bot: I haz the power, 2016-07-14 13:06:11]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-14 13:07:41]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-14 13:07:42]

Try jobs failed on following builders:
  v8_linux64_gyp_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng/build...)
  v8_linux_arm_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm_rel_ng/builds/...)

[Igor Sheludko, 2016-07-14 16:37:55]

The CQ bit was checked by ishell@chromium.org

[Igor Sheludko, 2016-07-14 16:37:56]

The patchset sent to the CQ was uploaded after l-g-t-m from
jkummerow@chromium.org
Link to the patchset: https://codereview.chromium.org/2147043002/#ps60001
(title: "Rebasing")

[commit-bot: I haz the power, 2016-07-14 16:37:59]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-14 17:04:28]

Description was changed from

==========
Cleanup interface descriptors to reflect that vectors are part of stores.

All stores require a Slot parameter so VectorStoreICTrampolineDescriptor is a
new StoreDescriptor.

VectorStoreICDescriptor renamed to StoreWithVectorDescriptor.
==========

to

==========
Cleanup interface descriptors to reflect that vectors are part of stores.

All stores require a Slot parameter so VectorStoreICTrampolineDescriptor is a
new StoreDescriptor.

VectorStoreICDescriptor renamed to StoreWithVectorDescriptor.
==========

[commit-bot: I haz the power, 2016-07-14 17:04:29]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-07-14 17:04:31]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-14 17:07:16]

Description was changed from

==========
Cleanup interface descriptors to reflect that vectors are part of stores.

All stores require a Slot parameter so VectorStoreICTrampolineDescriptor is a
new StoreDescriptor.

VectorStoreICDescriptor renamed to StoreWithVectorDescriptor.
==========

to

==========
Cleanup interface descriptors to reflect that vectors are part of stores.

All stores require a Slot parameter so VectorStoreICTrampolineDescriptor is a
new StoreDescriptor.

VectorStoreICDescriptor renamed to StoreWithVectorDescriptor.

Committed: https://crrev.com/53be25303970574b7d7ee3e3cd417c615d21dac2
Cr-Commit-Position: refs/heads/master@{#37779}
==========

[commit-bot: I haz the power, 2016-07-14 17:07:17]

Patchset 4 (id:??) landed as
https://crrev.com/53be25303970574b7d7ee3e3cd417c615d21dac2
Cr-Commit-Position: refs/heads/master@{#37779}