Refactoring to use ArrayVector where applicable

Refactoring to use ArrayVector where applicable

It's more readable than the construction
Vector<T>(buffer, arraysize(buffer)).
All those places are now replaced by ArrayVector(buffer).

R=titzer@chromium.org, jarin@chromium.org, rossberg@chromium.org

Committed: https://crrev.com/2f1df8a39fef897c1adfa6ce70787361ceb5a567
Cr-Commit-Position: refs/heads/master@{#35843}

[Clemens Hammacher, 2016-04-26 14:42:38]

If you think this is not reasonable, I am also fine with dropping this CL ;)

[titzer, 2016-04-26 14:53:35]

On 2016/04/26 14:42:38, Clemens Hammacher wrote:
> If you think this is not reasonable, I am also fine with dropping this CL ;)

lgtm

[Clemens Hammacher, 2016-04-27 08:08:23]

The CQ bit was checked by clemensh@chromium.org

[commit-bot: I haz the power, 2016-04-27 08:08:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1916393002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1916393002/1

[commit-bot: I haz the power, 2016-04-27 08:12:17]

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

[commit-bot: I haz the power, 2016-04-27 08:12:18]

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

[Clemens Hammacher, 2016-04-27 08:21:32]

Description was changed from

==========
Refactoring to use ArrayVector where applicable

It's more readable than the construction
Vector<T>(buffer, arraysize(buffer)).
All those places are now replaced by ArrayVector(buffer).

R=titzer@chromium.org
==========

to

==========
Refactoring to use ArrayVector where applicable

It's more readable than the construction
Vector<T>(buffer, arraysize(buffer)).
All those places are now replaced by ArrayVector(buffer).

R=titzer@chromium.org, jarin@chromium.org, rossberg@chromium.org
==========

[Clemens Hammacher, 2016-04-27 08:21:32]

clemensh@chromium.org changed reviewers:
+ jarin@chromium.org, rossberg@chromium.org

[Clemens Hammacher, 2016-04-27 08:23:07]

+rossberg for src/parsing
+jarin for src/crankshaft

[Jarin, 2016-04-27 08:50:00]

lgtm

[adamk, 2016-04-27 18:30:38]

adamk@chromium.org changed reviewers:
+ adamk@chromium.org

[adamk, 2016-04-27 18:30:39]

lgtm for parsing

[Clemens Hammacher, 2016-04-28 07:57:08]

The CQ bit was checked by clemensh@chromium.org

[Clemens Hammacher, 2016-04-28 07:57:08]

The patchset sent to the CQ was uploaded after l-g-t-m from titzer@chromium.org
Link to the patchset: https://codereview.chromium.org/1916393002/#ps20001
(title: "also refactor src/builtins.cc")

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

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1916393002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1916393002/20001

[commit-bot: I haz the power, 2016-04-28 08:00:00]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-01 22:01:47]

Description was changed from

==========
Refactoring to use ArrayVector where applicable

It's more readable than the construction
Vector<T>(buffer, arraysize(buffer)).
All those places are now replaced by ArrayVector(buffer).

R=titzer@chromium.org, jarin@chromium.org, rossberg@chromium.org
==========

to

==========
Refactoring to use ArrayVector where applicable

It's more readable than the construction
Vector<T>(buffer, arraysize(buffer)).
All those places are now replaced by ArrayVector(buffer).

R=titzer@chromium.org, jarin@chromium.org, rossberg@chromium.org

Committed: https://crrev.com/2f1df8a39fef897c1adfa6ce70787361ceb5a567
Cr-Commit-Position: refs/heads/master@{#35843}
==========

[commit-bot: I haz the power, 2016-05-01 22:01:48]

Patchset 2 (id:??) landed as
https://crrev.com/2f1df8a39fef897c1adfa6ce70787361ceb5a567
Cr-Commit-Position: refs/heads/master@{#35843}