Disable API call inline in TF when gather runtime stats

Disable API call inline in TF when gather runtime stats

BUG=none
R=epertoso@chromium.org,mvstanton@chromium.org,cbruni@chromium.org

Review-Url: https://codereview.chromium.org/2588363002
Cr-Commit-Position: refs/heads/master@{#41854}
Committed: https://chromium.googlesource.com/v8/v8/+/55c88e586badc6b29215c9a068900df13ea88721

[epertoso, 2016-12-20 11:41:41]

lgtm

[jochen (gone - plz use gerrit), 2016-12-20 11:44:38]

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

[commit-bot: I haz the power, 2016-12-20 11:44:48]

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

[Camillo Bruni, 2016-12-20 11:52:02]

lgtm

[mvstanton, 2016-12-20 11:55:31]

lgtm

[jochen (gone - plz use gerrit), 2016-12-20 11:55:55]

The CQ bit was unchecked by jochen@chromium.org

[jochen (gone - plz use gerrit), 2016-12-20 11:55:57]

The CQ bit was checked by jochen@chromium.org

[commit-bot: I haz the power, 2016-12-20 11:56:05]

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-12-20 12:10:50]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482234957704630, "parent_rev":
"fb43021511676963898c3d166d2d921632deb5d0", "commit_rev":
"55c88e586badc6b29215c9a068900df13ea88721"}

[commit-bot: I haz the power, 2016-12-20 12:10:53]

Description was changed from

==========
Disable API call inline in TF when gather runtime stats

BUG=none
R=epertoso@chromium.org,mvstanton@chromium.org,cbruni@chromium.org
==========

to

==========
Disable API call inline in TF when gather runtime stats

BUG=none
R=epertoso@chromium.org,mvstanton@chromium.org,cbruni@chromium.org

Review-Url: https://codereview.chromium.org/2588363002
Cr-Commit-Position: refs/heads/master@{#41854}
Committed:
https://chromium.googlesource.com/v8/v8/+/55c88e586badc6b29215c9a068900df13ea...
==========

[commit-bot: I haz the power, 2016-12-20 12:10:54]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/55c88e586badc6b29215c9a068900df13ea...

[Toon Verwaest, 2017-01-10 13:34:57]

verwaest@chromium.org changed reviewers:
+ verwaest@chromium.org

[Toon Verwaest, 2017-01-10 13:34:57]

https://codereview.chromium.org/2588363002/diff/1/src/compiler/js-native-cont...
File src/compiler/js-native-context-specialization.cc (right):

https://codereview.chromium.org/2588363002/diff/1/src/compiler/js-native-cont...
src/compiler/js-native-context-specialization.cc:1025: } else {
This is broken; if CanInlineApiCall returns false due to --runtime-call-stats,
you won't have a JSFunction and the code below can only deal with JSFunctions.