[typedarrays] check byte offset for fast typedarray sort

[typedarrays] check byte offset for fast typedarray sort

BUG=v8:5953
Review-Url: https://codereview.chromium.org/2733393002
Cr-Commit-Position: refs/heads/master@{#43666}
Committed: https://chromium.googlesource.com/v8/v8/+/5fc1bd5b32be6812c8e09d2486a81eb7d0918a1c

[Choongwoo Han, 2017-03-08 08:08:33]

Description was changed from

==========
[typedarrays] check byte offset for fast typedarray sort

BUG=v8:5953
==========

to

==========
[typedarrays] check byte offset for fast typedarray sort

BUG=v8:5953
==========

[Choongwoo Han, 2017-03-08 08:08:33]

cwhan.tunz@gmail.com changed reviewers:
+ bmeurer@chromium.org, caitp@igalia.com, petermarshall@chromium.org

[Choongwoo Han, 2017-03-08 08:11:02]

PTAL.

I found a bug from the patch I submitted before. Sorry for too many bugs :(. the
fast sorting also modified another elements which is located in back of
byte_offset.

[Benedikt Meurer, 2017-03-08 09:20:09]

lgtm

[Choongwoo Han, 2017-03-08 10:30:27]

The CQ bit was checked by cwhan.tunz@gmail.com to run a CQ dry run

[commit-bot: I haz the power, 2017-03-08 10:30:36]

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-03-08 11:01:48]

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

[commit-bot: I haz the power, 2017-03-08 11:01:49]

Dry run: This issue passed the CQ dry run.

[Choongwoo Han, 2017-03-08 11:08:16]

The CQ bit was checked by cwhan.tunz@gmail.com

[commit-bot: I haz the power, 2017-03-08 11:08:20]

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-03-08 12:01:13]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1488971296153690,
"parent_rev": "938e06f68fe9d055ee5af36e0c722856b2b60b36", "commit_rev":
"5afa697183cf40d2cd00d2e35582c8a0f8477e2c"}

[commit-bot: I haz the power, 2017-03-08 12:01:19]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1488971296153690,
"parent_rev": "efd4b2d3af8e122c76d05e692416a4e71d10de67", "commit_rev":
"5fc1bd5b32be6812c8e09d2486a81eb7d0918a1c"}

[commit-bot: I haz the power, 2017-03-08 12:01:24]

Description was changed from

==========
[typedarrays] check byte offset for fast typedarray sort

BUG=v8:5953
==========

to

==========
[typedarrays] check byte offset for fast typedarray sort

BUG=v8:5953

Review-Url: https://codereview.chromium.org/2733393002
Cr-Commit-Position: refs/heads/master@{#43666}
Committed:
https://chromium.googlesource.com/v8/v8/+/5fc1bd5b32be6812c8e09d2486a81eb7d09...
==========

[commit-bot: I haz the power, 2017-03-08 12:01:25]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/v8/v8/+/5fc1bd5b32be6812c8e09d2486a81eb7d09...