Re: [Mesa-dev] [PATCH 05/12] i965/cs: Split out helper for building local id payload

2015-10-08 Thread Pohjolainen, Topi
On Wed, Oct 07, 2015 at 07:11:45AM -0700, Kristian H?gsberg Kristensen wrote: > The initial motivation for this patch was to avoid calling > brw_cs_prog_local_id_payload_dwords() in gen7_cs_state.c from the > compiler. This commit ends up refactoring things a bit more so as to > split out the

Re: [Mesa-dev] [PATCH 05/12] i965/cs: Split out helper for building local id payload

2015-10-08 Thread Pohjolainen, Topi
On Wed, Oct 07, 2015 at 07:11:45AM -0700, Kristian H?gsberg Kristensen wrote: > The initial motivation for this patch was to avoid calling > brw_cs_prog_local_id_payload_dwords() in gen7_cs_state.c from the > compiler. This commit ends up refactoring things a bit more so as to > split out the

Re: [Mesa-dev] [PATCH 05/12] i965/cs: Split out helper for building local id payload

2015-10-08 Thread Kristian Høgsberg
On Thu, Oct 8, 2015 at 1:34 AM, Pohjolainen, Topi wrote: > On Wed, Oct 07, 2015 at 07:11:45AM -0700, Kristian H?gsberg Kristensen wrote: >> The initial motivation for this patch was to avoid calling >> brw_cs_prog_local_id_payload_dwords() in gen7_cs_state.c from the

Re: [Mesa-dev] [PATCH 05/12] i965/cs: Split out helper for building local id payload

2015-10-08 Thread Kristian Høgsberg
On Thu, Oct 8, 2015 at 1:53 AM, Pohjolainen, Topi wrote: > On Wed, Oct 07, 2015 at 07:11:45AM -0700, Kristian H?gsberg Kristensen wrote: >> The initial motivation for this patch was to avoid calling >> brw_cs_prog_local_id_payload_dwords() in gen7_cs_state.c from the

Re: [Mesa-dev] [PATCH 05/12] i965/cs: Split out helper for building local id payload

2015-10-08 Thread Pohjolainen, Topi
On Thu, Oct 08, 2015 at 10:53:59AM -0700, Kristian H?gsberg wrote: > On Thu, Oct 8, 2015 at 1:53 AM, Pohjolainen, Topi > wrote: > > On Wed, Oct 07, 2015 at 07:11:45AM -0700, Kristian H?gsberg Kristensen > > wrote: > >> The initial motivation for this patch was to

[Mesa-dev] [PATCH 05/12] i965/cs: Split out helper for building local id payload

2015-10-07 Thread Kristian Høgsberg Kristensen
The initial motivation for this patch was to avoid calling brw_cs_prog_local_id_payload_dwords() in gen7_cs_state.c from the compiler. This commit ends up refactoring things a bit more so as to split out the logic to build the local id payload to brw_fs.cpp. This moves the payload building closer