Changeset 86968 in webkit


Ignore:
Timestamp:
May 20, 2011 11:20:07 AM (13 years ago)
Author:
oliver@apple.com
Message:

2011-05-20 Oliver Hunt <oliver@apple.com>

Reviewed by Sam Weinig.

Remove unnecessary double->int conversion at the end of op_div
https://bugs.webkit.org/show_bug.cgi?id=61198

We don't attempt this conversion on 64bit, removing it actually speeds
up sunspider and v8 slightly, and it reduces code size.

  • jit/JITArithmetic32_64.cpp: (JSC::JIT::emit_op_div):
Location:
trunk/Source/JavaScriptCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/JavaScriptCore/ChangeLog

    r86965 r86968  
     12011-05-20  Oliver Hunt  <oliver@apple.com>
     2
     3        Reviewed by Sam Weinig.
     4
     5        Remove unnecessary double->int conversion at the end of op_div
     6        https://bugs.webkit.org/show_bug.cgi?id=61198
     7
     8        We don't attempt this conversion on 64bit, removing it actually speeds
     9        up sunspider and v8 slightly, and it reduces code size.
     10
     11        * jit/JITArithmetic32_64.cpp:
     12        (JSC::JIT::emit_op_div):
     13
    1142011-05-19  Evan Martin  <evan@chromium.org>
    215
  • trunk/Source/JavaScriptCore/jit/JITArithmetic32_64.cpp

    r82130 r86968  
    12471247    convertInt32ToDouble(regT2, fpRegT1);
    12481248    divDouble(fpRegT1, fpRegT0);
    1249 
    1250     JumpList doubleResult;
    1251     branchConvertDoubleToInt32(fpRegT0, regT0, doubleResult, fpRegT1);
    1252 
    1253     // Int32 result.
    1254     emitStoreInt32(dst, regT0, (op1 == dst || op2 == dst));
    1255     end.append(jump());
    1256 
    1257     // Double result.
    1258     doubleResult.link(this);
    12591249    emitStoreDouble(dst, fpRegT0);
    12601250    end.append(jump());
Note: See TracChangeset for help on using the changeset viewer.