[Webkit-unassigned] [Bug 45669] ARMv7Assembler generates illegal instructions with RVCT

[bugzilla-daemon at webkit.org]

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





--- Comment #8 from Gabor Loki <loki at webkit.org>  2010-09-14 01:24:30 PST ---
> The ctors were changed to zero the asInt value. (This follows similar convention used in this header file for other structs with bitfields) The storage on the asInt should take up 8 bits which corresponds to the 4 bits on lo4 and 4 bits on hi4. The problem is with the code that the type and amount only take up 7 bits.

Although I did not find any initialization problem in a small example (very similar to ShiftTypeAndAmount with the very same compiler), I feel this 4/4 and 2/5 bits difference could lead to a bug. So, it should be fixed.

I have one comment to your patch. It is really looks like a workaround. I suggest you try to fix it in less hacky way. For example:

- unsigned hi4() { return m_u.s1.hi4; }
+ unsigned hi4() { return m_u.s1.hi4 & 0x7; }

or

- unsigned hi4 : 4;
+ unsigned hi4 : 3;

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