[PATCH] wind3d: Acknowledge additional instruction modifiers

Henri Verbeet hverbeet at gmail.com
Sun Sep 9 08:33:15 CDT 2018


On 9 September 2018 at 17:37, Tobias Klausmann
<tobias.johannes.klausmann at mni.thm.de> wrote:
> i'm not sure which structure you really mean, the FIXME -> TRACE, or the
> bitextract instead of the bitmasks & shift
>
The latter, mostly. I think changing e.g.
AOFFIMMI_U_SHIFT/AOFFIMMI_U_MASK to
AOFFIMMI_U_BIT_LOW/AOFFIMMI_U_BIT_HIGH is gratuitous.

Looking a little more closely at the patch though, you're not really
parsing the RESOURCE_DIM and RESOURCE_RETURN_TYPE modifiers either, so
I think a FIXME would still be appropriate there. So both, I guess.



More information about the wine-devel mailing list