Re: [Mesa-dev] [PATCH 2/4] gallium: add threads per block TGSI property

2016-04-01 Thread Ilia Mirkin
On Fri, Apr 1, 2016 at 6:50 PM, Bas Nieuwenhuizen wrote: > I will change that to > > TGSI_PROPERTY_CS_FIXED_BLOCK_WIDTH etc. > > since most other properties, seem to use S instead of P, > unless you have any objections. Indeed they do - no objections from me. > > - Bas

Re: [Mesa-dev] [PATCH 2/4] gallium: add threads per block TGSI property

2016-04-01 Thread Bas Nieuwenhuizen
I will change that to TGSI_PROPERTY_CS_FIXED_BLOCK_WIDTH etc. since most other properties, seem to use S instead of P, unless you have any objections. - Bas On Sat, Apr 2, 2016 at 12:37 AM, Ilia Mirkin wrote: > On Fri, Apr 1, 2016 at 6:32 PM, Bas Nieuwenhuizen >

Re: [Mesa-dev] [PATCH 2/4] gallium: add threads per block TGSI property

2016-04-01 Thread Ilia Mirkin
On Fri, Apr 1, 2016 at 6:37 PM, Ilia Mirkin wrote: > On Fri, Apr 1, 2016 at 6:32 PM, Bas Nieuwenhuizen > wrote: >> The value 0 for unknown has been chosen to so that >> drivers using tgsi_scan_shader do not need to detect >> missing properties if

Re: [Mesa-dev] [PATCH 2/4] gallium: add threads per block TGSI property

2016-04-01 Thread Ilia Mirkin
On Fri, Apr 1, 2016 at 6:32 PM, Bas Nieuwenhuizen wrote: > The value 0 for unknown has been chosen to so that > drivers using tgsi_scan_shader do not need to detect > missing properties if they zero-initialize the struct. > > Signed-off-by: Bas Nieuwenhuizen

[Mesa-dev] [PATCH 2/4] gallium: add threads per block TGSI property

2016-04-01 Thread Bas Nieuwenhuizen
The value 0 for unknown has been chosen to so that drivers using tgsi_scan_shader do not need to detect missing properties if they zero-initialize the struct. Signed-off-by: Bas Nieuwenhuizen --- src/gallium/auxiliary/tgsi/tgsi_strings.c | 3 +++