<?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>133924</bug_id>
          
          <creation_ts>2014-06-14 23:05:59 -0700</creation_ts>
          <short_desc>Bracket access is significantly slower than property access</short_desc>
          <delta_ts>2014-06-14 23:23:46 -0700</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>Unspecified</rep_platform>
          <op_sys>Unspecified</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>Enhancement</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Ryosuke Niwa">rniwa</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>barraclough</cc>
    
    <cc>fpizlo</cc>
    
    <cc>kling</cc>
    
    <cc>mhahnenberg</cc>
    
    <cc>mike</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1015751</commentid>
    <comment_count>0</comment_count>
      <attachid>233135</attachid>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2014-06-14 23:05:59 -0700</bug_when>
    <thetext>Created attachment 233135
Microbenchmark

function getByVal(object, property) {
    return object[property];
}

function getById(object) {
    return object.myProperty;
}

The first function appears to be ~3.5x slower than the second function even if getByVal is always called as getByVal(object, &apos;myProperty&apos;).

It&apos;s only 1.5x slower on Chrome 35 and 2.2x slower on Firefox 30.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1015752</commentid>
    <comment_count>1</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2014-06-14 23:07:14 -0700</bug_when>
    <thetext>Note that it&apos;s 3.7x slower in the shipping Safari so it has gotten better.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1015753</commentid>
    <comment_count>2</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2014-06-14 23:07:27 -0700</bug_when>
    <thetext>(In reply to comment #1)
&gt; Note that it&apos;s 3.7x slower in the shipping Safari so it has gotten better.

4.7x*</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>233135</attachid>
            <date>2014-06-14 23:05:59 -0700</date>
            <delta_ts>2014-06-14 23:05:59 -0700</delta_ts>
            <desc>Microbenchmark</desc>
            <filename>getbyval-getbyid.html</filename>
            <type>text/html</type>
            <size>1052</size>
            <attacher name="Ryosuke Niwa">rniwa</attacher>
            
              <data encoding="base64">PCFET0NUWVBFIGh0bWw+CjxodG1sPgo8Ym9keT4KPHNjcmlwdD4KCndpbmRvdy5yZXN1bHQgPSBu
dWxsOwoKZnVuY3Rpb24gZ2V0QnlWYWwob2JqZWN0LCBwcm9wZXJ0eSkgewogICAgcmV0dXJuIG9i
amVjdFtwcm9wZXJ0eV07Cn0KCmZ1bmN0aW9uIGdldEJ5SWQob2JqZWN0KSB7CiAgICByZXR1cm4g
b2JqZWN0Lm15UHJvcGVydHk7Cn0KCmZ1bmN0aW9uIHJ1blRlc3QodGVzdEZ1bmN0aW9uKSB7CiAg
ICB2YXIgcmVzdWx0ID0gMDsKICAgIHZhciBzdGFydCA9IERhdGUubm93KCk7CiAgICB2YXIgb2Jq
ZWN0ID0ge215UHJvcGVydHk6IDEyM307CiAgICBmb3IgKHZhciBqID0gMDsgaiA8IDEwMDAgKiAx
MDAwOyBqKyspCiAgICAgICAgcmVzdWx0ICs9IHRlc3RGdW5jdGlvbihvYmplY3QsICdteVByb3Bl
cnR5Jyk7CiAgICByZXR1cm4gRGF0ZS5ub3coKSAtIHN0YXJ0Owp9Cgp2YXIgcmVzdWx0cyA9IFtd
OwoKZnVuY3Rpb24gY29tcGFyZShjb3VudCkgewogICAgdmFyIGRvY1Jlc3VsdCA9IHJ1blRlc3Qo
Z2V0QnlWYWwpOwogICAgdmFyIGRvY0VsZW1lbnRSZXN1bHQgPSBydW5UZXN0KGdldEJ5SWQpOwog
ICAgcmVzdWx0cy5wdXNoKGRvY1Jlc3VsdCAvIGRvY0VsZW1lbnRSZXN1bHQpOwogICAgZG9jdW1l
bnQuYm9keS5pbm5lckhUTUwgKz0gcmVzdWx0c1tyZXN1bHRzLmxlbmd0aCAtIDFdLnRvUHJlY2lz
aW9uKDQpICsgJzxicj4nOwoKICAgIGNvdW50LS07CiAgICBpZiAoY291bnQpCiAgICAgICAgc2V0
VGltZW91dChjb21wYXJlLmJpbmQodGhpcywgY291bnQpLCAxMDApOwogICAgZWxzZSB7CiAgICAg
ICAgZG9jdW1lbnQuYm9keS5pbm5lckhUTUwgKz0gJ0F2ZXJhZ2U6ICcKICAgICAgICAgICAgKyAo
cmVzdWx0cy5yZWR1Y2UoZnVuY3Rpb24gKGEsIGIpIHsgcmV0dXJuIGEgKyBiOyB9KSAvIHJlc3Vs
dHMubGVuZ3RoKS50b1ByZWNpc2lvbig0KTsKICAgIH0KfQoKc2V0VGltZW91dChjb21wYXJlLmJp
bmQodGhpcywgMjApLCAxMDApOwoKPC9zY3JpcHQ+Cjxmb3JtIG5hbWU9ImdldEVsZW1lbnRCeUlk
Ij48L2Zvcm0+CjwvYm9keT4KPC9odG1sPgo=
</data>

          </attachment>
      

    </bug>

</bugzilla>