[HEAP] Remove SIMD 128 bit alignment from heap.

[HEAP] Remove SIMD 128 bit alignment from heap.

LOG=N
BUG=v8:4124, v8:5948
Review-Url: https://codereview.chromium.org/2694063005
Cr-Commit-Position: refs/heads/master@{#43219}
Committed: https://chromium.googlesource.com/v8/v8/+/717c8f2c766fab803ea2ad37c4e1bd885f48ad90

[bbudge, 2017-02-14 18:24:29]

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

[commit-bot: I haz the power, 2017-02-14 18:24:38]

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-02-14 18:30:09]

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

[commit-bot: I haz the power, 2017-02-14 18:30:10]

Dry run: Try jobs failed on following builders:
  v8_win64_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win64_rel_ng/builds/22582)

[bbudge, 2017-02-14 19:14:14]

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

[commit-bot: I haz the power, 2017-02-14 19:14:24]

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-02-14 19:42:38]

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

[commit-bot: I haz the power, 2017-02-14 19:42:39]

Dry run: This issue passed the CQ dry run.

[bbudge, 2017-02-14 21:13:28]

bbudge@chromium.org changed reviewers:
+ mlippautz@chromium.org

[Yang, 2017-02-14 21:31:59]

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

[Yang, 2017-02-14 21:32:00]

https://codereview.chromium.org/2694063005/diff/20001/src/snapshot/deserializ...
File src/snapshot/deserializer.h (left):

https://codereview.chromium.org/2694063005/diff/20001/src/snapshot/deserializ...
src/snapshot/deserializer.h:87: DCHECK_LE(alignment, kSimd128Unaligned);
please do not remove this check, but rather change the upper bound to
kDoubleUnaligned.

[bbudge, 2017-02-14 22:18:09]

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

[commit-bot: I haz the power, 2017-02-14 22:18:20]

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

[bbudge, 2017-02-14 22:18:51]

https://codereview.chromium.org/2694063005/diff/20001/src/snapshot/deserializ...
File src/snapshot/deserializer.h (left):

https://codereview.chromium.org/2694063005/diff/20001/src/snapshot/deserializ...
src/snapshot/deserializer.h:87: DCHECK_LE(alignment, kSimd128Unaligned);
On 2017/02/14 21:32:00, Yang wrote:
> please do not remove this check, but rather change the upper bound to
> kDoubleUnaligned.

Good catch. Put it back.

[commit-bot: I haz the power, 2017-02-14 23:09:48]

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

[commit-bot: I haz the power, 2017-02-14 23:09:49]

Dry run: This issue passed the CQ dry run.

[Yang, 2017-02-15 09:53:40]

src/snapshot lgtm.

[Michael Lippautz, 2017-02-15 15:16:08]

lgtm

[bbudge, 2017-02-15 15:40:46]

The CQ bit was checked by bbudge@chromium.org

[commit-bot: I haz the power, 2017-02-15 15:40:49]

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-02-15 15:42:28]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1487173246202250,
"parent_rev": "67d087d577120a853d6b4c86c507236f0f45a423", "commit_rev":
"717c8f2c766fab803ea2ad37c4e1bd885f48ad90"}

[commit-bot: I haz the power, 2017-02-15 15:42:35]

Description was changed from

==========
[HEAP] Remove SIMD 128 bit alignment from heap.

LOG=N
BUG=v8:4124, v8:5948
==========

to

==========
[HEAP] Remove SIMD 128 bit alignment from heap.

LOG=N
BUG=v8:4124, v8:5948

Review-Url: https://codereview.chromium.org/2694063005
Cr-Commit-Position: refs/heads/master@{#43219}
Committed:
https://chromium.googlesource.com/v8/v8/+/717c8f2c766fab803ea2ad37c4e1bd885f4...
==========

[commit-bot: I haz the power, 2017-02-15 15:42:36]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/v8/v8/+/717c8f2c766fab803ea2ad37c4e1bd885f4...