<?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>129159</bug_id>
          
          <creation_ts>2014-02-21 11:39:09 -0800</creation_ts>
          <short_desc>Revive SABI (aka shouldAlwaysBeInlined)</short_desc>
          <delta_ts>2014-02-21 11:43:17 -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>RESOLVED</bug_status>
          <resolution>FIXED</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>
          
          <blocked>112840</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Filip Pizlo">fpizlo</reporter>
          <assigned_to name="Filip Pizlo">fpizlo</assigned_to>
          <cc>barraclough</cc>
    
    <cc>ggaren</cc>
    
    <cc>mark.lam</cc>
    
    <cc>mhahnenberg</cc>
    
    <cc>mmirman</cc>
    
    <cc>msaboff</cc>
    
    <cc>nrotem</cc>
    
    <cc>oliver</cc>
    
    <cc>sam</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>983268</commentid>
    <comment_count>0</comment_count>
    <who name="Filip Pizlo">fpizlo</who>
    <bug_when>2014-02-21 11:39:09 -0800</bug_when>
    <thetext>JSC has some cleverness for rapidly identifying functions that will always be inlined and so do not need to be separately optimized.  We broke this when we got rid of the baseline JIT inline caches and replaced them with the DFG ones.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>983269</commentid>
    <comment_count>1</comment_count>
      <attachid>224886</attachid>
    <who name="Filip Pizlo">fpizlo</who>
    <bug_when>2014-02-21 11:40:57 -0800</bug_when>
    <thetext>Created attachment 224886
the patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>983270</commentid>
    <comment_count>2</comment_count>
      <attachid>224886</attachid>
    <who name="Mark Hahnenberg">mhahnenberg</who>
    <bug_when>2014-02-21 11:42:04 -0800</bug_when>
    <thetext>Comment on attachment 224886
the patch

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>983271</commentid>
    <comment_count>3</comment_count>
    <who name="Filip Pizlo">fpizlo</who>
    <bug_when>2014-02-21 11:43:17 -0800</bug_when>
    <thetext>Landed in http://trac.webkit.org/changeset/164490</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>224886</attachid>
            <date>2014-02-21 11:40:57 -0800</date>
            <delta_ts>2014-02-21 11:42:04 -0800</delta_ts>
            <desc>the patch</desc>
            <filename>blah.patch</filename>
            <type>text/plain</type>
            <size>1610</size>
            <attacher name="Filip Pizlo">fpizlo</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291
cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gMTY0NDg5KQorKysgU291cmNl
L0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE1IEBA
CisyMDE0LTAyLTIxICBGaWxpcCBQaXpsbyAgPGZwaXpsb0BhcHBsZS5jb20+CisKKyAgICAgICAg
UmV2aXZlIFNBQkkgKGFrYSBzaG91bGRBbHdheXNCZUlubGluZWQpCisgICAgICAgIGh0dHBzOi8v
YnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMjkxNTkKKworICAgICAgICBSZXZpZXdl
ZCBieSBOT0JPRFkgKE9PUFMhKS4KKyAgICAgICAgCisgICAgICAgIFRoaXMgaXMgYSBzbWFsbCBP
Y3RhbmUgc3BlZWQtdXAuCisKKyAgICAgICAgKiBqaXQvUmVwYXRjaC5jcHA6CisgICAgICAgIChK
U0M6OmxpbmtGb3IpOiBUaGlzIGNvZGUgd2FzIGFzc3VtaW5nIHRoYXQgaWYgaXQncyBpbnZva2Vk
IHRoZW4gdGhlIGNhbGxlciBpcyBhIERGRyBjb2RlIGJsb2NrLiBUaGF0J3Mgd3JvbmcsIHNpbmNl
IGl0J3Mgbm93IHVzZWQgYnkgYWxsIG9mIHRoZSBKSVRzLgorCiAyMDE0LTAyLTIxICBKb3NlcGgg
UGVjb3Jhcm8gIDxwZWNvcmFyb0BhcHBsZS5jb20+CiAKICAgICAgICAgV2ViIEluc3BlY3Rvcjog
SlNDb250ZXh0IGluc3BlY3Rpb24gc2hvdWxkIHJlcG9ydCBleGNlcHRpb25zIGluIHRoZSBjb25z
b2xlCkluZGV4OiBTb3VyY2UvSmF2YVNjcmlwdENvcmUvaml0L1JlcGF0Y2guY3BwCj09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT0KLS0tIFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9qaXQvUmVwYXRjaC5jcHAJKHJldmlzaW9uIDE2
NDQ2NCkKKysrIFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9qaXQvUmVwYXRjaC5jcHAJKHdvcmtpbmcg
Y29weSkKQEAgLTE0MjMsMTEgKzE0MjMsMTIgQEAgdm9pZCBsaW5rRm9yKAogewogICAgIEFTU0VS
VCghY2FsbExpbmtJbmZvLnN0dWIpOwogICAgIAorICAgIENvZGVCbG9jayogY2FsbGVyQ29kZUJs
b2NrID0gZXhlYy0+Y2FsbGVyRnJhbWUoKS0+Y29kZUJsb2NrKCk7CisKICAgICAvLyBJZiB5b3Un
cmUgYmVpbmcgY2FsbC1saW5rZWQgZnJvbSBhIERGRyBjYWxsZXIgdGhlbiB5b3Ugb2J2aW91c2x5
IGRpZG4ndCBnZXQgaW5saW5lZC4KLSAgICBpZiAoY2FsbGVlQ29kZUJsb2NrKQorICAgIGlmIChj
YWxsZWVDb2RlQmxvY2sgJiYgSklUQ29kZTo6aXNPcHRpbWl6aW5nSklUKGNhbGxlckNvZGVCbG9j
ay0+aml0VHlwZSgpKSkKICAgICAgICAgY2FsbGVlQ29kZUJsb2NrLT5tX3Nob3VsZEFsd2F5c0Jl
SW5saW5lZCA9IGZhbHNlOwogICAgIAotICAgIENvZGVCbG9jayogY2FsbGVyQ29kZUJsb2NrID0g
ZXhlYy0+Y2FsbGVyRnJhbWUoKS0+Y29kZUJsb2NrKCk7CiAgICAgVk0qIHZtID0gY2FsbGVyQ29k
ZUJsb2NrLT52bSgpOwogICAgIAogICAgIFJlcGF0Y2hCdWZmZXIgcmVwYXRjaEJ1ZmZlcihjYWxs
ZXJDb2RlQmxvY2spOwo=
</data>
<flag name="review"
          id="248998"
          type_id="1"
          status="+"
          setter="mhahnenberg"
    />
          </attachment>
      

    </bug>

</bugzilla>