[builtins] Implement %TypedArray%.prototype.map in the CSA

[builtins] Implement %TypedArray%.prototype.map in the CSA

R=danno@chromium.org

Review-Url: https://codereview.chromium.org/2814683002
Cr-Commit-Position: refs/heads/master@{#45380}
Committed: https://chromium.googlesource.com/v8/v8/+/d4f80f4c0117456e87099498512ba51100cbdc79

[Camillo Bruni, 2017-04-11 13:16:05]

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

[Camillo Bruni, 2017-04-11 13:16:06]

https://codereview.chromium.org/2814683002/diff/60001/src/builtins/builtins-a...
File src/builtins/builtins-array-gen.cc (right):

https://codereview.chromium.org/2814683002/diff/60001/src/builtins/builtins-a...
src/builtins/builtins-array-gen.cc:194: Bind(&fast);
drive-by-comment: Could you upgrade your implementation to use the VARIABLE and
BIND macros? (I'll do a pass tomorrow to fix up the pending ones and unify the
interface)

[Camillo Bruni, 2017-04-11 13:16:17]

cbruni@chromium.org changed reviewers:
- cbruni@chromium.org

[Tobias Tebbi, 2017-04-20 15:59:03]

tebbi@chromium.org changed reviewers:
+ mvstanton@chromium.org
- danno@chromium.org

[Tobias Tebbi, 2017-04-20 16:00:02]

tebbi@chromium.org changed reviewers:
+ danno@chromium.org

[danno, 2017-04-24 16:11:16]

lgtm with comments

https://codereview.chromium.org/2814683002/diff/80001/src/builtins/builtins-a...
File src/builtins/builtins-array-gen.cc (right):

https://codereview.chromium.org/2814683002/diff/80001/src/builtins/builtins-a...
src/builtins/builtins-array-gen.cc:159: Node* A =
TypedArraySpeciesCreateByLength(context(), o(), len_);
super nit: elsewhere to conform to the coding style that we've been using for
this stuff so far, perhaps a lower-case "a"?

https://codereview.chromium.org/2814683002/diff/80001/src/objects-inl.h
File src/objects-inl.h (right):

https://codereview.chromium.org/2814683002/diff/80001/src/objects-inl.h#newco...
src/objects-inl.h:6923: Handle<Object> receiver) {
Nice cleanup.

[Tobias Tebbi, 2017-04-25 11:04:12]

The CQ bit was checked by tebbi@chromium.org

[Tobias Tebbi, 2017-04-25 11:04:13]

The patchset sent to the CQ was uploaded after l-g-t-m from danno@chromium.org
Link to the patchset: https://codereview.chromium.org/2814683002/#ps120001
(title: "Rebased.")

[commit-bot: I haz the power, 2017-04-25 11:04:18]

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-04-25 11:15:08]

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

[commit-bot: I haz the power, 2017-04-25 11:15:09]

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

[Tobias Tebbi, 2017-05-17 14:50:07]

The CQ bit was checked by tebbi@chromium.org

[Tobias Tebbi, 2017-05-17 14:50:07]

The patchset sent to the CQ was uploaded after l-g-t-m from danno@chromium.org
Link to the patchset: https://codereview.chromium.org/2814683002/#ps220001
(title: "added todo")

[commit-bot: I haz the power, 2017-05-17 14:50:17]

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-05-17 15:27:38]

CQ is committing da patch.

Bot data: {"patchset_id": 220001, "attempt_start_ts": 1495032607543780,
"parent_rev": "f1e82a2ee9d339f86b47cd4507c0a58f9c06e608", "commit_rev":
"d4f80f4c0117456e87099498512ba51100cbdc79"}

[commit-bot: I haz the power, 2017-05-17 15:27:46]

Description was changed from

==========
[builtins] Implement %TypedArray%.prototype.map in the CSA

R=danno@chromium.org
==========

to

==========
[builtins] Implement %TypedArray%.prototype.map in the CSA

R=danno@chromium.org

Review-Url: https://codereview.chromium.org/2814683002
Cr-Commit-Position: refs/heads/master@{#45380}
Committed:
https://chromium.googlesource.com/v8/v8/+/d4f80f4c0117456e87099498512ba51100c...
==========

[commit-bot: I haz the power, 2017-05-17 15:27:47]

Committed patchset #12 (id:220001) as
https://chromium.googlesource.com/v8/v8/+/d4f80f4c0117456e87099498512ba51100c...