https://bugs.kde.org/show_bug.cgi?id=370028

--- Comment #40 from Ivo Raisr <iv...@ivosh.net> ---
(In reply to Tamara Vlahovic from comment #38)

I've had a look at patch for x87 stuff. While passing Fpu_State* instead of
UChar* is a good idea, on the other hand changing UChar[80] to UShort[40] is
not a very good idead, though. As you are aware, the real x87 layout contains 8
registers per 80 bits (10 bytes). So it is hard to express 80 bits as a natural
C type; and that is the reason why UChar[80] was chosen.
In addition to that, Fpu_State_16 still contains UChar[80], leading to
confusion.

I don't know if there is a good/straightforward way how to mitigate warnings
for reg[80]. Perhaps some kind of union? If you are not sure, then split the
patch and I will take in Fpu_State* stuff while you can work on reg[80] part.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to