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 <kenn...@whitecape.org>
Reviewed-by: Kristian Høgsberg <k...@bitplanet.net>
> ---
> 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
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
