Changeset 193983 in webkit


Ignore:
Timestamp:
Dec 11, 2015 2:39:29 PM (8 years ago)
Author:
keith_miller@apple.com
Message:

Overrides has instance should not move ValueFalse to a register then immediately to the stack in the LLInt.
https://bugs.webkit.org/show_bug.cgi?id=152188

Reviewed by Mark Lam.

This fixes a minor issue with the code for the overrides_has_instance in the LLInt. Old code had an extra move,
which is both slow and breaks the build on cloop.

  • llint/LowLevelInterpreter64.asm:
Location:
trunk/Source/JavaScriptCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/JavaScriptCore/ChangeLog

    r193974 r193983  
     12015-12-11  Keith Miller  <keith_miller@apple.com>
     2
     3        Overrides has instance should not move ValueFalse to a register then immediately to the stack in the LLInt.
     4        https://bugs.webkit.org/show_bug.cgi?id=152188
     5
     6        Reviewed by Mark Lam.
     7
     8        This fixes a minor issue with the code for the overrides_has_instance in the LLInt. Old code had an extra move,
     9        which is both slow and breaks the build on cloop.
     10
     11        * llint/LowLevelInterpreter64.asm:
     12
    1132015-12-11  Keith Miller  <keith_miller@apple.com>
    214
  • trunk/Source/JavaScriptCore/llint/LowLevelInterpreter64.asm

    r193974 r193983  
    10891089
    10901090.opOverridesHasInstanceNotDefaultSymbol:
    1091     storeq ValueTrue, t1
    1092     storeq t1, [cfr, t3, 8]
     1091    storeq ValueTrue, [cfr, t3, 8]
    10931092    dispatch(4)
    10941093
Note: See TracChangeset for help on using the changeset viewer.