Clean-up aligned allocation logic.

Clean-up aligned allocation logic.

BUG=

Committed: https://crrev.com/055d4ee19cf12d0ec590b0d9e992a610cc62cd6c
Cr-Commit-Position: refs/heads/master@{#28430}

[Hannes Payer (out of office), 2015-05-15 15:06:52]

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

[jochen (gone - plz use gerrit), 2015-05-15 15:21:11]

https://codereview.chromium.org/1138643005/diff/1/src/heap/heap.cc
File src/heap/heap.cc (right):

https://codereview.chromium.org/1138643005/diff/1/src/heap/heap.cc#newcode2178
src/heap/heap.cc:2178: heap->old_space()->AllocateRaw(object_size,
kDoubleAligned);
same here?

https://codereview.chromium.org/1138643005/diff/1/src/heap/mark-compact.cc
File src/heap/mark-compact.cc (right):

https://codereview.chromium.org/1138643005/diff/1/src/heap/mark-compact.cc#ne...
src/heap/mark-compact.cc:1955: allocation = new_space->AllocateRaw(size,
kDoubleAligned);
why not the same logic as above?

[Hannes Payer (out of office), 2015-05-15 15:41:01]

https://codereview.chromium.org/1138643005/diff/1/src/heap/heap.cc
File src/heap/heap.cc (right):

https://codereview.chromium.org/1138643005/diff/1/src/heap/heap.cc#newcode2178
src/heap/heap.cc:2178: heap->old_space()->AllocateRaw(object_size,
kDoubleAligned);
On 2015/05/15 15:21:11, jochen (traveling) wrote:
> same here?

same copy-paste. Done.

https://codereview.chromium.org/1138643005/diff/1/src/heap/mark-compact.cc
File src/heap/mark-compact.cc (right):

https://codereview.chromium.org/1138643005/diff/1/src/heap/mark-compact.cc#ne...
src/heap/mark-compact.cc:1955: allocation = new_space->AllocateRaw(size,
kDoubleAligned);
On 2015/05/15 15:21:11, jochen (traveling) wrote:
> why not the same logic as above?

copy-paste... Done.

[jochen (gone - plz use gerrit), 2015-05-15 15:43:51]

lgtm

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

The CQ bit was checked by hpayer@chromium.org

[commit-bot: I haz the power, 2015-05-15 15:44:20]

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

[commit-bot: I haz the power, 2015-05-15 17:04:54]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-15 17:05:07]

Patchset 2 (id:??) landed as
https://crrev.com/055d4ee19cf12d0ec590b0d9e992a610cc62cd6c
Cr-Commit-Position: refs/heads/master@{#28430}