[Webkit-unassigned] [Bug 52222] Microoptimization in ~JSString

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=52222


Xan Lopez <xan.lopez at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #78544|                            |review?
               Flag|                            |




--- Comment #1 from Xan Lopez <xan.lopez at gmail.com>  2011-01-11 09:49:41 PST ---
Created an attachment (id=78544)
 --> (https://bugs.webkit.org/attachment.cgi?id=78544&action=review)
jsstringdtor.diff

Small optimization in the JSString dtor. SunSpider results:


TEST                   COMPARISON            FROM                 TO             DETAILS

=============================================================================

** TOTAL **:           1.003x as fast    275.4ms +/- 0.1%   274.7ms +/- 0.1%     significant

=============================================================================

  3d:                  -                  36.4ms +/- 0.5%    36.2ms +/- 0.5% 
    cube:              -                  11.9ms +/- 1.0%    11.8ms +/- 1.0% 
    morph:             -                  11.2ms +/- 0.6%    11.2ms +/- 0.6% 
    raytrace:          -                  13.2ms +/- 0.5%    13.2ms +/- 0.5% 

  access:              -                  38.7ms +/- 0.2%    38.7ms +/- 0.2% 
    binary-trees:      ??                  3.7ms +/- 2.0%     3.7ms +/- 2.0%     not conclusive: might be *1.002x as slow*
    fannkuch:          -                  19.0ms +/- 0.1%    19.0ms +/- 0.0% 
    nbody:             -                  11.0ms +/- 0.0%    11.0ms +/- 0.0% 
    nsieve:            -                   5.0ms +/- 0.0%     5.0ms +/- 0.0% 

  bitops:              ??                 23.6ms +/- 0.3%    23.7ms +/- 0.3%     not conclusive: might be *1.002x as slow*
    3bit-bits-in-byte: -                   3.0ms +/- 0.0%     3.0ms +/- 0.0% 
    bits-in-byte:      -                   6.0ms +/- 0.0%     6.0ms +/- 0.0% 
    bitwise-and:       ??                  5.6ms +/- 1.4%     5.7ms +/- 1.4%     not conclusive: might be *1.008x as slow*
    nsieve-bits:       -                   9.0ms +/- 0.1%     9.0ms +/- 0.0% 

  controlflow:         -                   2.0ms +/- 0.0%     2.0ms +/- 0.0% 
    recursive:         -                   2.0ms +/- 0.0%     2.0ms +/- 0.0% 

  crypto:              -                  16.0ms +/- 0.1%    16.0ms +/- 0.1% 
    aes:               -                  10.0ms +/- 0.0%    10.0ms +/- 0.0% 
    md5:               -                   3.0ms +/- 0.6%     3.0ms +/- 0.6% 
    sha1:              -                   3.0ms +/- 0.0%     3.0ms +/- 0.0% 

  date:                ??                 31.4ms +/- 0.3%    31.4ms +/- 0.3%     not conclusive: might be *1.001x as slow*
    format-tofte:      -                  16.0ms +/- 0.2%    16.0ms +/- 0.2% 
    format-xparb:      ??                 15.3ms +/- 0.5%    15.4ms +/- 0.6%     not conclusive: might be *1.003x as slow*

  math:                -                  29.0ms +/- 0.1%    29.0ms +/- 0.1% 
    cordic:            ??                  9.0ms +/- 0.3%     9.0ms +/- 0.4%     not conclusive: might be *1.001x as slow*
    partial-sums:      -                  15.0ms +/- 0.0%    15.0ms +/- 0.1% 
    spectral-norm:     -                   5.0ms +/- 0.0%     5.0ms +/- 0.0% 

  regexp:              ??                 12.5ms +/- 0.6%    12.6ms +/- 0.6%     not conclusive: might be *1.005x as slow*
    dna:               ??                 12.5ms +/- 0.6%    12.6ms +/- 0.6%     not conclusive: might be *1.005x as slow*

  string:              1.008x as fast     85.8ms +/- 0.1%    85.1ms +/- 0.1%     significant
    base64:            -                   9.0ms +/- 0.1%     9.0ms +/- 0.0% 
    fasta:             1.052x as fast     12.0ms +/- 0.2%    11.4ms +/- 0.7%     significant
    tagcloud:          -                  21.8ms +/- 0.3%    21.7ms +/- 0.3% 
    unpack-code:       -                  32.0ms +/- 0.1%    32.0ms +/- 0.1% 
    validate-input:    -                  11.0ms +/- 0.2%    11.0ms +/- 0.2%

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.