Remove explicit double alignment from allocation helper functions.

Remove explicit double alignment from allocation helper functions.

BUG=

Committed: https://crrev.com/250f591c94c234a27b8e94330cfdb3240feda677
Cr-Commit-Position: refs/heads/master@{#28320}

[Hannes Payer (out of office), 2015-05-08 11:44:39]

hpayer@chromium.org changed reviewers:
+ jochen@chromium.org

[Hannes Payer (out of office), 2015-05-08 11:45:42]

hpayer@chromium.org changed reviewers:
+ bmeurer@chromium.org

[Benedikt Meurer, 2015-05-08 11:47:43]

https://codereview.chromium.org/1128323003/diff/1/src/heap/heap.h
File src/heap/heap.h (right):

https://codereview.chromium.org/1128323003/diff/1/src/heap/heap.h#newcode1824
src/heap/heap.h:1824: bool double_align = false);
Please don't use bool. Add an enum AlignmentConstraint {kWordAlign,kDoubleAlign}
or something like that.

[Hannes Payer (out of office), 2015-05-08 11:59:25]

https://codereview.chromium.org/1128323003/diff/1/src/heap/heap.h
File src/heap/heap.h (right):

https://codereview.chromium.org/1128323003/diff/1/src/heap/heap.h#newcode1824
src/heap/heap.h:1824: bool double_align = false);
On 2015/05/08 11:47:42, Benedikt Meurer wrote:
> Please don't use bool. Add an enum AlignmentConstraint
{kWordAlign,kDoubleAlign}
> or something like that.

Done.

[Benedikt Meurer, 2015-05-08 12:01:31]

LGTM once comment addressed

https://codereview.chromium.org/1128323003/diff/20001/src/heap/heap.h
File src/heap/heap.h (right):

https://codereview.chromium.org/1128323003/diff/20001/src/heap/heap.h#newcode...
src/heap/heap.h:1818: enum Alignment { kWordAligned, kDoubleWordAligned };
s/kDoubleWordAligned/kDoubleAligned

[Hannes Payer (out of office), 2015-05-08 12:04:30]

https://codereview.chromium.org/1128323003/diff/20001/src/heap/heap.h
File src/heap/heap.h (right):

https://codereview.chromium.org/1128323003/diff/20001/src/heap/heap.h#newcode...
src/heap/heap.h:1818: enum Alignment { kWordAligned, kDoubleWordAligned };
On 2015/05/08 12:01:31, Benedikt Meurer wrote:
> s/kDoubleWordAligned/kDoubleAligned

Done.

[Hannes Payer (out of office), 2015-05-08 13:24:27]

The CQ bit was checked by hpayer@chromium.org

[Hannes Payer (out of office), 2015-05-08 13:24:28]

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

[commit-bot: I haz the power, 2015-05-08 13:24:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1128323003/40001

[commit-bot: I haz the power, 2015-05-08 14:11:53]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-05-08 14:12:12]

Patchset 3 (id:??) landed as
https://crrev.com/250f591c94c234a27b8e94330cfdb3240feda677
Cr-Commit-Position: refs/heads/master@{#28320}