[ic] Drop TimerEventScopes from *IC_Miss runtime functions

[ic] Drop TimerEventScopes from *IC_Miss runtime functions

This is mostly a performance experiment. If it provides no speedup,
it can be reverted to keep IC miss events in timeline plots.
Otherwise, the RuntimeCallStats system is the replacement tool for
investigating performance issues related to IC misses.

This effectively reverts 1f8adc15 / r21736.

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

[Jakob Kummerow, 2016-11-07 16:59:50]

jkummerow@chromium.org changed reviewers:
+ yangguo@chromium.org

[Jakob Kummerow, 2016-11-07 16:59:51]

PTAL.

https://codereview.chromium.org/2480343002/diff/1/test/mjsunit/tools/profviz-...
File test/mjsunit/tools/profviz-test.default (right):

https://codereview.chromium.org/2480343002/diff/1/test/mjsunit/tools/profviz-...
test/mjsunit/tools/profviz-test.default:451: "set object 439 rect from 22.2675,
18.83 to 22.3815, 18.17 fc rgb \"#00CC00\"",
This is just s/19/18/ and so on.

[Yang, 2016-11-08 10:11:08]

On 2016/11/07 16:59:51, Jakob Kummerow wrote:
> PTAL.
> 
>
https://codereview.chromium.org/2480343002/diff/1/test/mjsunit/tools/profviz-...
> File test/mjsunit/tools/profviz-test.default (right):
> 
>
https://codereview.chromium.org/2480343002/diff/1/test/mjsunit/tools/profviz-...
> test/mjsunit/tools/profviz-test.default:451: "set object 439 rect from
22.2675,
> 18.83 to 22.3815, 18.17 fc rgb \"#00CC00\"",
> This is just s/19/18/ and so on.

lgtm.

[Jakob Kummerow, 2016-11-08 10:12:28]

The CQ bit was checked by jkummerow@chromium.org

[commit-bot: I haz the power, 2016-11-08 10:12:38]

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-11-08 10:28:06]

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

[commit-bot: I haz the power, 2016-11-08 10:28:07]

Try jobs failed on following builders:
  v8_linux64_gyp_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng/build...)
  v8_linux64_gyp_rel_ng_triggered on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng_trigg...)

[Jakob Kummerow, 2016-11-10 12:41:09]

The CQ bit was checked by jkummerow@chromium.org

[Jakob Kummerow, 2016-11-10 12:41:09]

The patchset sent to the CQ was uploaded after l-g-t-m from yangguo@chromium.org
Link to the patchset: https://codereview.chromium.org/2480343002/#ps20001
(title: "drop debugging leftover")

[commit-bot: I haz the power, 2016-11-10 12:41:15]

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-11-10 13:10:12]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-17 22:29:13]

Description was changed from

==========
[ic] Drop TimerEventScopes from *IC_Miss runtime functions

This is mostly a performance experiment. If it provides no speedup,
it can be reverted to keep IC miss events in timeline plots.
Otherwise, the RuntimeCallStats system is the replacement tool for
investigating performance issues related to IC misses.

This effectively reverts 1f8adc15 / r21736.
==========

to

==========
[ic] Drop TimerEventScopes from *IC_Miss runtime functions

This is mostly a performance experiment. If it provides no speedup,
it can be reverted to keep IC miss events in timeline plots.
Otherwise, the RuntimeCallStats system is the replacement tool for
investigating performance issues related to IC misses.

This effectively reverts 1f8adc15 / r21736.

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

[commit-bot: I haz the power, 2016-11-17 22:29:14]

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