On Wed, Aug 27, 2014 at 3:10 PM, Ian Romanick <i...@freedesktop.org> wrote:
> On 08/27/2014 02:30 PM, Jordan Justen wrote:
>> diff --git a/src/mesa/drivers/dri/i965/brw_context.h
>> b/src/mesa/drivers/dri/i965/brw_context.h
>> index 602275c..7475135 100644
>> --- a/src/mesa/drivers/dri/i965/brw_context.h
>> +++ b/src/mesa/drivers/dri/i965/brw_context.h
>> @@ -241,6 +241,13 @@ struct brw_state_flags {
>> GLuint cache;
>> };
>>
>> +
>> +/**
>> + * Set one of the bits in a field of brw_state_flags.
>> + */
>> +#define SET_DIRTY_BIT(FIELD, FLAG) brw->state.dirty.FIELD |= (FLAG)
>
> I think this should get the do { ... } while(0) treatment.
Paul added it in patch 4. That work?
-Jordan
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
