Changeset 159153 in webkit


Ignore:
Timestamp:
Nov 12, 2013 3:09:39 PM (10 years ago)
Author:
commit-queue@webkit.org
Message:

[sh4] Fix load32WithUnalignedHalfWords function in baseline JIT.
https://bugs.webkit.org/show_bug.cgi?id=124233

Patch by Julien Brianceau <jbriance@cisco.com> on 2013-11-12
Reviewed by Michael Saboff.

  • assembler/MacroAssemblerSH4.h:

(JSC::MacroAssemblerSH4::load32WithUnalignedHalfWords): Do not claim scratch register too early.
Test already covered by fast/regex/pcre-test-1.

Location:
trunk/Source/JavaScriptCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/JavaScriptCore/ChangeLog

    r159141 r159153  
     12013-11-12  Julien Brianceau  <jbriance@cisco.com>
     2
     3        [sh4] Fix load32WithUnalignedHalfWords function in baseline JIT.
     4        https://bugs.webkit.org/show_bug.cgi?id=124233
     5
     6        Reviewed by Michael Saboff.
     7
     8        * assembler/MacroAssemblerSH4.h:
     9        (JSC::MacroAssemblerSH4::load32WithUnalignedHalfWords): Do not claim scratch register too early.
     10        Test already covered by fast/regex/pcre-test-1.
     11
    1122013-11-12  Filip Pizlo  <fpizlo@apple.com>
    213
  • trunk/Source/JavaScriptCore/assembler/MacroAssemblerSH4.h

    r158580 r159153  
    13151315    {
    13161316        RegisterID scr = claimScratch();
    1317         RegisterID scr1 = claimScratch();
    13181317        Jump m_jump;
    13191318        JumpList end;
    13201319
     1320        loadEffectiveAddress(address, scr);
     1321
     1322        RegisterID scr1 = claimScratch();
    13211323        if (dest != SH4Registers::r0)
    13221324            move(SH4Registers::r0, scr1);
    1323 
    1324         loadEffectiveAddress(address, scr);
    13251325
    13261326        m_assembler.ensureSpace(m_assembler.maxInstructionSize + 58, sizeof(uint32_t));
Note: See TracChangeset for help on using the changeset viewer.