Generalize alignment in heap GC functions.

Generalize alignment in heap GC functions.
Changes template parameters from int to AllocationAlignment.

LOG=N
BUG=v8:4124
Committed: https://crrev.com/871ab7f7b1847bdb78773b8abfec2b65c2b1faa8
Cr-Commit-Position: refs/heads/master@{#28587}

[bbudge, 2015-05-21 12:27:43]

Patchset #1 (id:1) has been deleted

[bbudge, 2015-05-21 12:28:19]

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

[Hannes Payer (out of office), 2015-05-22 06:11:13]

https://codereview.chromium.org/1152513002/diff/20001/src/heap/heap.cc
File src/heap/heap.cc (right):

https://codereview.chromium.org/1152513002/diff/20001/src/heap/heap.cc#newcod...
src/heap/heap.cc:2257: #define kObjectAligned kDoubleAligned
Why do we need this?

https://codereview.chromium.org/1152513002/diff/20001/src/heap/heap.cc#newcod...
src/heap/heap.cc:2265: EvacuateObject<POINTER_OBJECT, kObjectAligned>(map, slot,
object,
Why not kWordAligned?

[bbudge, 2015-05-22 07:34:33]

https://codereview.chromium.org/1152513002/diff/20001/src/heap/heap.cc
File src/heap/heap.cc (right):

https://codereview.chromium.org/1152513002/diff/20001/src/heap/heap.cc#newcod...
src/heap/heap.cc:2257: #define kObjectAligned kDoubleAligned
On 2015/05/22 06:11:12, Hannes Payer wrote:
> Why do we need this?

I was confused about word size vs. word aligned, which I think means "aligned to
pointer size." Deleted.

https://codereview.chromium.org/1152513002/diff/20001/src/heap/heap.cc#newcod...
src/heap/heap.cc:2265: EvacuateObject<POINTER_OBJECT, kObjectAligned>(map, slot,
object,
On 2015/05/22 06:11:13, Hannes Payer wrote:
> Why not kWordAligned?

Changed to use kWordAligned.

[Hannes Payer (out of office), 2015-05-22 10:50:31]

Cool, thx. LGTM

[bbudge, 2015-05-22 11:22:23]

The CQ bit was checked by bbudge@chromium.org

[commit-bot: I haz the power, 2015-05-22 11:22:30]

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

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

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2015-05-22 12:02:16]

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