[interpreter] Also optimize small functions earlier.

[interpreter] Also optimize small functions earlier.

For fullcodegen the RuntimeProfiler has a shortcut that allows it to
tier up small functions earlier, when enough type feedback is available.
Port the same optimization for the Ignition+TurboFan pipeline.

R=mstarzinger@chromium.org

Committed: https://crrev.com/b888150afd6f5d83f4a110315bea6c9ca72fd912
Cr-Commit-Position: refs/heads/master@{#40435}

[Benedikt Meurer, 2016-10-19 12:00:33]

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

[commit-bot: I haz the power, 2016-10-19 12:00:40]

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

[Benedikt Meurer, 2016-10-19 12:01:25]

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

[Benedikt Meurer, 2016-10-19 12:01:26]

Hey Michi, Mythri,

Here's a port of the FCG functionality. Fixes some of the slowness we see in
SunSpider, probably needs tweaking.
Please take a look.

Thanks,
Benedikt

[Michael Starzinger, 2016-10-19 12:08:25]

LGTM.

[Benedikt Meurer, 2016-10-19 12:27:03]

The CQ bit was unchecked by bmeurer@chromium.org

[Benedikt Meurer, 2016-10-19 12:27:11]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2016-10-19 12:27:21]

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, 2016-10-19 13:11:55]

Committed patchset #1 (id:1)

[mythria, 2016-10-19 13:17:30]

lgtm with one comment more for my understanding.

https://codereview.chromium.org/2427283004/diff/1/src/runtime-profiler.cc
File src/runtime-profiler.cc (right):

https://codereview.chromium.org/2427283004/diff/1/src/runtime-profiler.cc#new...
src/runtime-profiler.cc:429: // If no IC was patched since the last tick and
this function is very
Just for my understanding, any_ic_changed_ means if the feedback has changed
right? I am not sure if any changes in the Call feedback or binary op feedback
are Notified to runtime-profiler from ignition.  Would it be important to notify
these changes? If it is important should we add a TODO here?

[Benedikt Meurer, 2016-10-19 13:19:41]

https://codereview.chromium.org/2427283004/diff/1/src/runtime-profiler.cc
File src/runtime-profiler.cc (right):

https://codereview.chromium.org/2427283004/diff/1/src/runtime-profiler.cc#new...
src/runtime-profiler.cc:429: // If no IC was patched since the last tick and
this function is very
On 2016/10/19 13:17:29, mythria wrote:
> Just for my understanding, any_ic_changed_ means if the feedback has changed
> right? I am not sure if any changes in the Call feedback or binary op feedback
> are Notified to runtime-profiler from ignition.  Would it be important to
notify
> these changes? If it is important should we add a TODO here?

That's a good point, I missed that. Can you put that on the agenda for your next
discussion with Ross?

[commit-bot: I haz the power, 2016-11-17 22:06:31]

Description was changed from

==========
[interpreter] Also optimize small functions earlier.

For fullcodegen the RuntimeProfiler has a shortcut that allows it to
tier up small functions earlier, when enough type feedback is available.
Port the same optimization for the Ignition+TurboFan pipeline.

R=mstarzinger@chromium.org
==========

to

==========
[interpreter] Also optimize small functions earlier.

For fullcodegen the RuntimeProfiler has a shortcut that allows it to
tier up small functions earlier, when enough type feedback is available.
Port the same optimization for the Ignition+TurboFan pipeline.

R=mstarzinger@chromium.org

Committed: https://crrev.com/b888150afd6f5d83f4a110315bea6c9ca72fd912
Cr-Commit-Position: refs/heads/master@{#40435}
==========

[commit-bot: I haz the power, 2016-11-17 22:06:32]

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