Re: [Mesa-dev] [PATCH 03/25] amd/common: cleanup DATA_FORMAT/NUM_FORMAT field names

2018-12-19 Thread Haehnle, Nicolai
On 17.12.18 04:23, Marek Olšák wrote: > The definitions weren't changed, but the values were. The names need to > be different, so that si_debug.c prints both the GFX6 and GFX9 values. You're right. I have some larger changes of the debug printing that is smarter about which fields to print

Re: [Mesa-dev] [PATCH 03/25] amd/common: cleanup DATA_FORMAT/NUM_FORMAT field names

2018-12-16 Thread Marek Olšák
The definitions weren't changed, but the values were. The names need to be different, so that si_debug.c prints both the GFX6 and GFX9 values. Marek On Thu, Dec 6, 2018 at 9:01 AM Nicolai Hähnle wrote: > From: Nicolai Hähnle > > The definition wasn't actually changed in gfx9, so having the

[Mesa-dev] [PATCH 03/25] amd/common: cleanup DATA_FORMAT/NUM_FORMAT field names

2018-12-06 Thread Nicolai Hähnle
From: Nicolai Hähnle The definition wasn't actually changed in gfx9, so having the suffix makes no sense. --- src/amd/common/ac_nir_to_llvm.c | 2 +- src/amd/common/gfx9d.h| 12 ++-- src/amd/common/sid.h | 12