[Webkit-unassigned] [Bug 112886] [SH4] LLInt sh4 backend implementation

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=112886





--- Comment #8 from Julien Brianceau <jbrianceau at nds.com>  2013-03-22 09:09:18 PST ---
Another thing that I do not understand inside risc.rb is the riscLowerShiftOps, where:
# lshifti foo, bar
#
# will become:
#
# andi foo, 31, tmp
# lshifti tmp, bar

This potentially adds a useless opcode (when foo < 32), and worst, this can lead to an incorrect behaviour. For instance:

lshifti 33, register

will become

and 33, 31, tmp
lshifti tmp, register

So we'll left-shift register by 1 (because 33 & 31 = 1), and I don't think this is what we expect here, right?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.