<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>145503</bug_id>
          
          <creation_ts>2015-05-31 11:02:28 -0700</creation_ts>
          <short_desc>Explore turning off inlining of functions that haven&apos;t made it to Baseline</short_desc>
          <delta_ts>2017-12-01 10:31:32 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>JavaScriptCore</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>NEW</bug_status>
          <resolution></resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Filip Pizlo">fpizlo</reporter>
          <assigned_to name="Robin Morisset">rmorisset</assigned_to>
          <cc>rmorisset</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1098416</commentid>
    <comment_count>0</comment_count>
    <who name="Filip Pizlo">fpizlo</who>
    <bug_when>2015-05-31 11:02:28 -0700</bug_when>
    <thetext>...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1376893</commentid>
    <comment_count>1</comment_count>
    <who name="Robin Morisset">rmorisset</who>
    <bug_when>2017-12-01 10:31:32 -0800</bug_when>
    <thetext>I have started experimenting with this.
It provided a very small, barely significant (when lucky and there is very low noise in the benchmark) speed-up of about 0.5% on the old jsc-only benchmarks (Octane, Kraken, Sunspider).

I would like to do more thorough benchmarking, including ARES/Speedometer before landing this change.

I have also tested scaling the inliningCost by the tier reached by the callee (encouraging inlining of very hot functions, and discouraging inlining of very cold ones). So far it has not seemed to bring any measurable additional improvement.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>