[heap] Slightly optimize Heap::UpdateAllocationSite()

[heap] Slightly optimize Heap::UpdateAllocationSite()

This function is called for every live object in new space.  We statically know
which version to call, so let's use templates here and eliminite a branch at
runtime.

BUG=chromium:524425
LOG=N

Committed: https://crrev.com/062fc9abcf0528e77d88ee9bf7b66b799e94d10f
Cr-Commit-Position: refs/heads/master@{#33662}

[Michael Lippautz, 2016-02-02 08:34:38]

Description was changed from

==========
[heap] Slightly optimize Heap::UpdateAllocationSite()

We statically know which version to call, so let's use templates here and
eliminite a branch at runtime.

BUG=chromium:524425
LOG=N
==========

to

==========
[heap] Slightly optimize Heap::UpdateAllocationSite()

This function is called for every live object in new space.
We statically know which version to call, so let's use templates here and
eliminite a branch at runtime.

BUG=chromium:524425
LOG=N
==========

[Michael Lippautz, 2016-02-02 08:34:51]

Description was changed from

==========
[heap] Slightly optimize Heap::UpdateAllocationSite()

This function is called for every live object in new space.
We statically know which version to call, so let's use templates here and
eliminite a branch at runtime.

BUG=chromium:524425
LOG=N
==========

to

==========
[heap] Slightly optimize Heap::UpdateAllocationSite()

This function is called for every live object in new space.  We statically know
which version to call, so let's use templates here and eliminite a branch at
runtime.

BUG=chromium:524425
LOG=N
==========

[Michael Lippautz, 2016-02-02 08:35:41]

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

[commit-bot: I haz the power, 2016-02-02 08:36:47]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1659823002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1659823002/1

[commit-bot: I haz the power, 2016-02-02 09:03:37]

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

[commit-bot: I haz the power, 2016-02-02 09:03:38]

Dry run: This issue passed the CQ dry run.

[Michael Lippautz, 2016-02-02 09:15:35]

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

[Michael Lippautz, 2016-02-02 09:15:35]

https://codereview.chromium.org/1659823002/diff/1/src/heap/heap.h
File src/heap/heap.h (left):

https://codereview.chromium.org/1659823002/diff/1/src/heap/heap.h#oldcode451
src/heap/heap.h:451: enum PretenuringFeedbackInsertionMode { kCached, kGlobal };
We already had an enum (dead code) for that purpose ;)

[ulan, 2016-02-02 09:24:55]

lgtm

[Michael Lippautz, 2016-02-02 09:25:49]

The CQ bit was checked by mlippautz@chromium.org

[commit-bot: I haz the power, 2016-02-02 09:25:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1659823002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1659823002/1

[commit-bot: I haz the power, 2016-02-02 09:28:10]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-02 09:29:00]

Description was changed from

==========
[heap] Slightly optimize Heap::UpdateAllocationSite()

This function is called for every live object in new space.  We statically know
which version to call, so let's use templates here and eliminite a branch at
runtime.

BUG=chromium:524425
LOG=N
==========

to

==========
[heap] Slightly optimize Heap::UpdateAllocationSite()

This function is called for every live object in new space.  We statically know
which version to call, so let's use templates here and eliminite a branch at
runtime.

BUG=chromium:524425
LOG=N

Committed: https://crrev.com/062fc9abcf0528e77d88ee9bf7b66b799e94d10f
Cr-Commit-Position: refs/heads/master@{#33662}
==========

[commit-bot: I haz the power, 2016-02-02 09:29:01]

Patchset 1 (id:??) landed as
https://crrev.com/062fc9abcf0528e77d88ee9bf7b66b799e94d10f
Cr-Commit-Position: refs/heads/master@{#33662}