<?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>154131</bug_id>
          
          <creation_ts>2016-02-11 14:09:33 -0800</creation_ts>
          <short_desc>[JSC] SqrtFloat and CeilFloat also suffer from partial register stalls</short_desc>
          <delta_ts>2016-02-11 15:01:43 -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>New Bugs</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</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>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Benjamin Poulain">benjamin</reporter>
          <assigned_to name="Benjamin Poulain">benjamin</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>keith_miller</cc>
    
    <cc>mark.lam</cc>
    
    <cc>msaboff</cc>
    
    <cc>saam</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1163864</commentid>
    <comment_count>0</comment_count>
    <who name="Benjamin Poulain">benjamin</who>
    <bug_when>2016-02-11 14:09:33 -0800</bug_when>
    <thetext>[JSC] SqrtFloat and CeilFloat also suffer from partial register stalls</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1163866</commentid>
    <comment_count>1</comment_count>
      <attachid>271084</attachid>
    <who name="Benjamin Poulain">benjamin</who>
    <bug_when>2016-02-11 14:10:02 -0800</bug_when>
    <thetext>Created attachment 271084
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1163891</commentid>
    <comment_count>2</comment_count>
    <who name="Benjamin Poulain">benjamin</who>
    <bug_when>2016-02-11 14:43:05 -0800</bug_when>
    <thetext>Looks like it actually helped ASM: 

                                                  Conf#1                    Conf#2                                      
Octane:
   encrypt                                   0.26580+-0.00535          0.26261+-0.00181         might be 1.0121x faster
   decrypt                                   4.88253+-0.03087          4.87441+-0.03265       
   deltablue                        x2       0.22707+-0.02065    ?     0.23526+-0.01640       ? might be 1.0360x slower
   earley                                    0.50553+-0.00370          0.50526+-0.00190       
   boyer                                     8.29859+-0.03093          8.29258+-0.14059       
   navier-stokes                    x2       6.32582+-0.01449    ?     6.32637+-0.01641       ?
   raytrace                         x2       1.51112+-0.02407          1.51109+-0.01574       
   richards                         x2       0.13826+-0.00187    ?     0.14065+-0.00125       ? might be 1.0173x slower
   splay                            x2       0.53466+-0.01274          0.53390+-0.00423       
   regexp                           x2      38.63601+-0.54019         38.61920+-0.35350       
   pdfjs                            x2      60.58862+-0.72967    ?    60.71452+-0.21419       ?
   mandreel                         x2      70.06093+-0.31377    ?    70.42351+-1.25984       ?
   gbemu                            x2      45.88083+-0.18418    ?    46.13054+-0.10402       ?
   closure                                   0.93782+-0.00608          0.93723+-0.00767       
   jquery                                   11.81939+-0.05536         11.77315+-0.10492       
   box2d                            x2      16.68482+-0.14566         16.68110+-0.14917       
   zlib                             x2     581.25326+-2.14803        578.19639+-3.10252       
   typescript                       x2    1116.63348+-8.15026    ?  1127.10419+-16.04267      ?

   &lt;geometric&gt;                               8.58508+-0.05871    ?     8.61839+-0.05741       ? might be 1.0039x slower

                                                  Conf#1                    Conf#2                                      
Kraken:
   ai-astar                                  149.728+-1.075      ^     147.221+-0.765         ^ definitely 1.0170x faster
   audio-beat-detection                       74.380+-0.512      ?      74.519+-0.516         ?
   audio-dft                                 128.216+-0.377            127.508+-1.973         
   audio-fft                                  60.330+-0.390      ?      60.382+-1.155         ?
   audio-oscillator                           85.936+-0.186      ?      86.273+-1.178         ?
   imaging-darkroom                           96.946+-0.359      ?      96.993+-0.440         ?
   imaging-desaturate                         84.382+-0.666             84.331+-0.158         
   imaging-gaussian-blur                     125.591+-0.756            125.500+-0.345         
   json-parse-financial                       67.640+-0.446             67.623+-0.546         
   json-stringify-tinderbox                   40.490+-0.455      ?      40.679+-0.380         ?
   stanford-crypto-aes                        62.713+-1.357      ?      63.494+-0.359         ? might be 1.0125x slower
   stanford-crypto-ccm                        59.512+-2.354      ?      61.230+-5.146         ? might be 1.0289x slower
   stanford-crypto-pbkdf2                    152.271+-0.684      ?     152.522+-2.581         ?
   stanford-crypto-sha256-iterative           59.230+-0.361      ?      59.251+-0.775         ?

   &lt;arithmetic&gt;                               89.098+-0.145      ?      89.109+-0.502         ? might be 1.0001x slower

                                                  Conf#1                    Conf#2                                      
AsmBench:
   bigfib.cpp                               655.1461+-20.9275    ?    665.8336+-4.7698        ? might be 1.0163x slower
   cray.c                                   608.0889+-9.1078          606.2350+-6.4679        
   dry.c                                    812.7330+-462.8484        745.3911+-318.6448        might be 1.0903x faster
   FloatMM.c                                925.2051+-64.5055         924.6276+-65.0364       
   gcc-loops.cpp                           6285.7328+-27.1328        6280.3089+-24.3046       
   n-body.c                                1615.9001+-30.6388        1604.6730+-13.7856       
   Quicksort.c                              599.0604+-4.2295          598.6930+-1.7205        
   stepanov_container.cpp                  4484.4255+-74.5501        4472.9373+-49.4676       
   Towers.c                                 384.6421+-0.7442     ?    384.9072+-0.6414        ?

   &lt;geometric&gt;                             1130.8891+-54.3618        1122.3520+-45.4478         might be 1.0076x faster

                                                  Conf#1                    Conf#2                                      
Geomean of preferred means:
   &lt;scaled-result&gt;                           95.2746+-1.5452           95.1637+-1.4671          might be 1.0012x faster</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1163898</commentid>
    <comment_count>3</comment_count>
      <attachid>271084</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2016-02-11 15:01:40 -0800</bug_when>
    <thetext>Comment on attachment 271084
Patch

Clearing flags on attachment: 271084

Committed r196444: &lt;http://trac.webkit.org/changeset/196444&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1163899</commentid>
    <comment_count>4</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2016-02-11 15:01:43 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>271084</attachid>
            <date>2016-02-11 14:10:02 -0800</date>
            <delta_ts>2016-02-11 15:01:40 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-154131-20160211140941.patch</filename>
            <type>text/plain</type>
            <size>1466</size>
            <attacher name="Benjamin Poulain">benjamin</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTk2NDM2CmRpZmYgLS1naXQgYS9Tb3VyY2UvSmF2YVNjcmlw
dENvcmUvQ2hhbmdlTG9nIGIvU291cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwppbmRleCAz
NDhlMzMyZDRmNWMyODA3ZWUxMzY2NzdmM2UzYzNkOGM0MmRiYzJmLi45ZmI1MTQ5ZTU0M2Y3NGM0
NDA0Y2Y1N2RmMTYzOTg0YWRkMzBkZWExIDEwMDY0NAotLS0gYS9Tb3VyY2UvSmF2YVNjcmlwdENv
cmUvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKQEAgLTEs
MyArMSwxNSBAQAorMjAxNi0wMi0xMSAgQmVuamFtaW4gUG91bGFpbiAgPGJlbmphbWluQHdlYmtp
dC5vcmc+CisKKyAgICAgICAgW0pTQ10gU3FydEZsb2F0IGFuZCBDZWlsRmxvYXQgYWxzbyBzdWZm
ZXIgZnJvbSBwYXJ0aWFsIHJlZ2lzdGVyIHN0YWxscworICAgICAgICBodHRwczovL2J1Z3Mud2Vi
a2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTU0MTMxCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9C
T0RZIChPT1BTISkuCisKKyAgICAgICAgTG9va3MgbGlrZSBJIGZvcmdvdCB0byB1cGRhdGUgdGhp
cyB3aGVuIGFkZGluZyBGbG9hdCBzdXBwb3J0LgorICAgICAgICBDcmVkaXQgdG8gRmlsaXAgZm9y
IGZpbmRpbmcgdGhpcyBpc3N1ZS4KKworICAgICAgICAqIGIzL2Fpci9BaXJGaXhQYXJ0aWFsUmVn
aXN0ZXJTdGFsbHMuY3BwOgorCiAyMDE2LTAyLTExICBTdWtvbHNhayBTYWtzaHV3b25nICA8c3Vr
b2xzYWtAZ21haWwuY29tPgogCiAgICAgICAgIFtJTlRMXSBJbXBsZW1lbnQgSW50bC5OdW1iZXJG
b3JtYXQucHJvdG90eXBlLnJlc29sdmVkT3B0aW9ucyAoKQpkaWZmIC0tZ2l0IGEvU291cmNlL0ph
dmFTY3JpcHRDb3JlL2IzL2Fpci9BaXJGaXhQYXJ0aWFsUmVnaXN0ZXJTdGFsbHMuY3BwIGIvU291
cmNlL0phdmFTY3JpcHRDb3JlL2IzL2Fpci9BaXJGaXhQYXJ0aWFsUmVnaXN0ZXJTdGFsbHMuY3Bw
CmluZGV4IDc1ODI0MGE0Zjc4OGMyZTU0ZWUwODkwNDViODY0ZWY2N2Y5YWYxYjUuLmI0ODkzMDhk
ZDhhNDcwMzE2ZDYzMzNlMjg1ZWI0ZTIyNDg2ZmE4NTUgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9KYXZh
U2NyaXB0Q29yZS9iMy9haXIvQWlyRml4UGFydGlhbFJlZ2lzdGVyU3RhbGxzLmNwcAorKysgYi9T
b3VyY2UvSmF2YVNjcmlwdENvcmUvYjMvYWlyL0FpckZpeFBhcnRpYWxSZWdpc3RlclN0YWxscy5j
cHAKQEAgLTUxLDcgKzUxLDkgQEAgYm9vbCBoYXNQYXJ0aWFsWG1tUmVnVXBkYXRlKGNvbnN0IElu
c3QmIGluc3QpCiAgICAgY2FzZSBDb252ZXJ0SW50MzJUb0RvdWJsZToKICAgICBjYXNlIENvbnZl
cnRJbnQ2NFRvRG91YmxlOgogICAgIGNhc2UgU3FydERvdWJsZToKKyAgICBjYXNlIFNxcnRGbG9h
dDoKICAgICBjYXNlIENlaWxEb3VibGU6CisgICAgY2FzZSBDZWlsRmxvYXQ6CiAgICAgICAgIHJl
dHVybiB0cnVlOwogICAgIGRlZmF1bHQ6CiAgICAgICAgIGJyZWFrOwo=
</data>

          </attachment>
      

    </bug>

</bugzilla>