On Sat, Jun 28, 2014 at 09:33:48PM -0700, Kenneth Graunke wrote: > While we're adding things, use symbolic constants rather than magic > numbers. > > Signed-off-by: Kenneth Graunke <[email protected]>
Reviewed-by: Kristian Høgsberg <[email protected]> > --- > src/mesa/drivers/dri/i965/brw_disasm.c | 24 +++++++++++++----------- > 1 file changed, 13 insertions(+), 11 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_disasm.c > b/src/mesa/drivers/dri/i965/brw_disasm.c > index e40938a..26b60b6 100644 > --- a/src/mesa/drivers/dri/i965/brw_disasm.c > +++ b/src/mesa/drivers/dri/i965/brw_disasm.c > @@ -231,17 +231,19 @@ static const char *const pred_ctrl_align16[16] = { > }; > > static const char *const pred_ctrl_align1[16] = { > - [1] = "", > - [2] = ".anyv", > - [3] = ".allv", > - [4] = ".any2h", > - [5] = ".all2h", > - [6] = ".any4h", > - [7] = ".all4h", > - [8] = ".any8h", > - [9] = ".all8h", > - [10] = ".any16h", > - [11] = ".all16h", > + [BRW_PREDICATE_NORMAL] = "", > + [BRW_PREDICATE_ALIGN1_ANYV] = ".anyv", > + [BRW_PREDICATE_ALIGN1_ALLV] = ".allv", > + [BRW_PREDICATE_ALIGN1_ANY2H] = ".any2h", > + [BRW_PREDICATE_ALIGN1_ALL2H] = ".all2h", > + [BRW_PREDICATE_ALIGN1_ANY4H] = ".any4h", > + [BRW_PREDICATE_ALIGN1_ALL4H] = ".all4h", > + [BRW_PREDICATE_ALIGN1_ANY8H] = ".any8h", > + [BRW_PREDICATE_ALIGN1_ALL8H] = ".all8h", > + [BRW_PREDICATE_ALIGN1_ANY16H] = ".any16h", > + [BRW_PREDICATE_ALIGN1_ANY16H] = ".all16h", > + [BRW_PREDICATE_ALIGN1_ANY32H] = ".any32h", > + [BRW_PREDICATE_ALIGN1_ANY32H] = ".all32h", > }; > > static const char *const thread_ctrl[4] = { > -- > 2.0.0 > > _______________________________________________ > mesa-dev mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
