[Webkit-unassigned] [Bug 79104] Unaligned userspace access for SH4 platforms
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Mar 6 23:22:04 PST 2012
https://bugs.webkit.org/show_bug.cgi?id=79104
Filip Pizlo <fpizlo at apple.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #127968|review? |review-
Flag| |
--- Comment #3 from Filip Pizlo <fpizlo at apple.com> 2012-03-06 23:22:03 PST ---
(From update of attachment 127968)
View in context: https://bugs.webkit.org/attachment.cgi?id=127968&action=review
> Source/JavaScriptCore/assembler/MacroAssemblerSH4.h:978
> + m_assembler.ensureSpace(m_assembler.maxInstructionSize + 68, sizeof(uint32_t));
> + move(scr, SH4Registers::r0);
> + m_assembler.andlImm8r(0x3, SH4Registers::r0);
> + m_assembler.cmpEqImmR0(0x0, SH4Registers::r0);
> + m_assembler.branch(BF_OPCODE, 2 + extraInst);
> +
> + if (dest != SH4Registers::r0)
> + move(scr1, SH4Registers::r0);
> +
> + load32(scr, dest);
> + m_assembler.branch(BRA_OPCODE, 25 + extraInst);
> + m_assembler.nop();
> +
> + m_assembler.andlImm8r(0x1, SH4Registers::r0);
> + m_assembler.cmpEqImmR0(0x0, SH4Registers::r0);
> +
> + if (dest != SH4Registers::r0)
> + move(scr1, SH4Registers::r0);
> +
> + m_assembler.branch(BF_OPCODE, 8);
Why aren't you just using the macro assembler API instead of having to manually compute branch offsets?
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list