Changeset 190372 in webkit


Ignore:
Timestamp:
Sep 30, 2015, 4:27:44 PM (10 years ago)
Author:
keith_miller@apple.com
Message:

Relanding r190289 with the following two fixes:

  1. REGRESSION(r190289): It made Speedometer/Full.html performance test fail https://bugs.webkit.org/show_bug.cgi?id=149621

Reviewed by Saam Barati.

We need to restore callee saves for both the fast and slow paths before making a
tail call in the FTL.

  • ftl/FTLJSCallBase.cpp: (JSC::FTL::JSCallBase::emit):
  1. [ARM] REGRESSION(r190289): It made 374 tests crash on 32 bit ARM Linux https://bugs.webkit.org/show_bug.cgi?id=149619

Reviewed by Filip Pizlo.

Need to check for ARMv7_TRADITIONAL and ARMv7 in addition to ARM in "if"
statement to handle platforms with a link register.

  • llint/LowLevelInterpreter.asm: (prepareForTailCall):

Patch by Michael Saboff <msaboff@apple.com> on 2015-09-30

Location:
trunk/Source/JavaScriptCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/JavaScriptCore/CMakeLists.txt

    r190370 r190372  
    10901090    ${JAVASCRIPTCORE_DIR}/builtins/StringConstructor.js
    10911091    ${JAVASCRIPTCORE_DIR}/builtins/StringIterator.prototype.js
     1092    ${JAVASCRIPTCORE_DIR}/builtins/TypedArray.prototype.js
    10921093)
    10931094
  • trunk/Source/JavaScriptCore/ChangeLog

    r190370 r190372  
    2424            * llint/LowLevelInterpreter.asm:
    2525            (prepareForTailCall):
     26
     272015-09-30  Keith Miller  <keith_miller@apple.com>
     28
     29        Unreviewed Windows buildfix.
     30
     31        * CMakeLists.txt:
    2632
    27332015-09-30  Keith Miller  <keith_miller@apple.com>
Note: See TracChangeset for help on using the changeset viewer.