Enable emplace_back for zone containers

Enable emplace_back for zone containers

As required by C++11, this CL changes the zone allocator to be able to
construct and destroy arbitrary types, and accept arbitrary arguments
for construct, passing them via perfect forwarding.
I also change some push_back to emplace_back. Some of those did not
compile before.

R=ishell@chromium.org, titzer@chromium.org

Review-Url: https://codereview.chromium.org/2646873004
Cr-Commit-Position: refs/heads/master@{#42597}
Committed: https://chromium.googlesource.com/v8/v8/+/14f470cafd909c7c5805edc0257048f495491f66

[Clemens Hammacher, 2017-01-20 13:14:09]

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

[commit-bot: I haz the power, 2017-01-20 13:14:18]

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, 2017-01-20 13:50:12]

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

[commit-bot: I haz the power, 2017-01-20 13:50:12]

Dry run: This issue passed the CQ dry run.

[Clemens Hammacher, 2017-01-20 13:56:02]

Description was changed from

==========
Enable emplace_back for zone containers

As required by C++11, this CL changes the zone allocator to be able to
construct and destroy arbitrary types, and accept arbitrary arguments
for construct, passing them via perfect forwarding.
I also change some push_back to emplace_back. Some of those did not
compile before.
==========

to

==========
Enable emplace_back for zone containers

As required by C++11, this CL changes the zone allocator to be able to
construct and destroy arbitrary types, and accept arbitrary arguments
for construct, passing them via perfect forwarding.
I also change some push_back to emplace_back. Some of those did not
compile before.

R=ishell@chromium.org, titzer@chromium.org
==========

[Clemens Hammacher, 2017-01-20 13:56:02]

clemensh@chromium.org changed reviewers:
+ ishell@chromium.org, titzer@chromium.org

[titzer, 2017-01-20 14:02:52]

On 2017/01/20 13:58:10, Clemens Hammacher wrote:

lgtm. Seems like this will simplify a lot of push_backs().

[Clemens Hammacher, 2017-01-20 14:05:22]

On 2017/01/20 at 14:02:52, titzer wrote:
> On 2017/01/20 13:58:10, Clemens Hammacher wrote:
> 
> lgtm. Seems like this will simplify a lot of push_backs().

Yep, if people remember that this method exists ;)
It also increases performance since it might save a copy (at least for
non-trivially-copyable types).

[Igor Sheludko, 2017-01-23 10:35:55]

lgtm

[Clemens Hammacher, 2017-01-23 10:37:10]

The CQ bit was checked by clemensh@chromium.org

[commit-bot: I haz the power, 2017-01-23 10:37:13]

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, 2017-01-23 11:12:27]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485167830993150, "parent_rev":
"1f069202e11864e509f493ac13c571adcb7b5736", "commit_rev":
"14f470cafd909c7c5805edc0257048f495491f66"}

[commit-bot: I haz the power, 2017-01-23 11:12:32]

Description was changed from

==========
Enable emplace_back for zone containers

As required by C++11, this CL changes the zone allocator to be able to
construct and destroy arbitrary types, and accept arbitrary arguments
for construct, passing them via perfect forwarding.
I also change some push_back to emplace_back. Some of those did not
compile before.

R=ishell@chromium.org, titzer@chromium.org
==========

to

==========
Enable emplace_back for zone containers

As required by C++11, this CL changes the zone allocator to be able to
construct and destroy arbitrary types, and accept arbitrary arguments
for construct, passing them via perfect forwarding.
I also change some push_back to emplace_back. Some of those did not
compile before.

R=ishell@chromium.org, titzer@chromium.org

Review-Url: https://codereview.chromium.org/2646873004
Cr-Commit-Position: refs/heads/master@{#42597}
Committed:
https://chromium.googlesource.com/v8/v8/+/14f470cafd909c7c5805edc0257048f4954...
==========

[commit-bot: I haz the power, 2017-01-23 11:12:33]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/14f470cafd909c7c5805edc0257048f4954...