Re: [Mesa-dev] [PATCH 07/17] i965/fs: Move offset() and half() to the fs_builder

2015-06-24 Thread Jason Ekstrand
On Jun 24, 2015 4:29 AM, "Francisco Jerez" wrote: > > Jason Ekstrand writes: > > > On Tue, Jun 23, 2015 at 9:22 AM, Francisco Jerez wrote: > >> Jason Ekstrand writes: > >> > >>> We want to move these into the builder so that they know the current > >>> builder's dispatch width. This will be ne

Re: [Mesa-dev] [PATCH 07/17] i965/fs: Move offset() and half() to the fs_builder

2015-06-24 Thread Jason Ekstrand
On Wed, Jun 24, 2015 at 7:56 AM, Francisco Jerez wrote: > Jason Ekstrand writes: > >> On Wed, Jun 24, 2015 at 6:44 AM, Francisco Jerez >> wrote: >>> Jason Ekstrand writes: >>> On Jun 24, 2015 6:29 AM, "Francisco Jerez" wrote: > > Jason Ekstrand writes: > > > On Jun 24, 2

Re: [Mesa-dev] [PATCH 07/17] i965/fs: Move offset() and half() to the fs_builder

2015-06-24 Thread Francisco Jerez
Jason Ekstrand writes: > On Wed, Jun 24, 2015 at 6:44 AM, Francisco Jerez > wrote: >> Jason Ekstrand writes: >> >>> On Jun 24, 2015 6:29 AM, "Francisco Jerez" wrote: Jason Ekstrand writes: > On Jun 24, 2015 4:29 AM, "Francisco Jerez" >>> wrote: >> >> Jason Ekst

Re: [Mesa-dev] [PATCH 07/17] i965/fs: Move offset() and half() to the fs_builder

2015-06-24 Thread Jason Ekstrand
On Wed, Jun 24, 2015 at 6:44 AM, Francisco Jerez wrote: > Jason Ekstrand writes: > >> On Jun 24, 2015 6:29 AM, "Francisco Jerez" wrote: >>> >>> Jason Ekstrand writes: >>> >>> > On Jun 24, 2015 4:29 AM, "Francisco Jerez" >> wrote: >>> >> >>> >> Jason Ekstrand writes: >>> >> >>> >> > On Tue, Ju

Re: [Mesa-dev] [PATCH 07/17] i965/fs: Move offset() and half() to the fs_builder

2015-06-24 Thread Francisco Jerez
Jason Ekstrand writes: > On Jun 24, 2015 6:29 AM, "Francisco Jerez" wrote: >> >> Jason Ekstrand writes: >> >> > On Jun 24, 2015 4:29 AM, "Francisco Jerez" > wrote: >> >> >> >> Jason Ekstrand writes: >> >> >> >> > On Tue, Jun 23, 2015 at 9:22 AM, Francisco Jerez < > curroje...@riseup.net> >> >

Re: [Mesa-dev] [PATCH 07/17] i965/fs: Move offset() and half() to the fs_builder

2015-06-24 Thread Francisco Jerez
Jason Ekstrand writes: > On Jun 24, 2015 4:29 AM, "Francisco Jerez" wrote: >> >> Jason Ekstrand writes: >> >> > On Tue, Jun 23, 2015 at 9:22 AM, Francisco Jerez > wrote: >> >> Jason Ekstrand writes: >> >> >> >>> We want to move these into the builder so that they know the current >> >>> build

Re: [Mesa-dev] [PATCH 07/17] i965/fs: Move offset() and half() to the fs_builder

2015-06-24 Thread Francisco Jerez
Jason Ekstrand writes: > On Tue, Jun 23, 2015 at 9:22 AM, Francisco Jerez > wrote: >> Jason Ekstrand writes: >> >>> We want to move these into the builder so that they know the current >>> builder's dispatch width. This will be needed by a later commit. >> >> I very much like the idea of this

Re: [Mesa-dev] [PATCH 07/17] i965/fs: Move offset() and half() to the fs_builder

2015-06-23 Thread Jason Ekstrand
On Tue, Jun 23, 2015 at 1:39 AM, Pohjolainen, Topi wrote: > On Thu, Jun 18, 2015 at 05:51:36PM -0700, Jason Ekstrand wrote: >> We want to move these into the builder so that they know the current >> builder's dispatch width. This will be needed by a later commit. >> --- >> src/mesa/drivers/dri/i

Re: [Mesa-dev] [PATCH 07/17] i965/fs: Move offset() and half() to the fs_builder

2015-06-23 Thread Jason Ekstrand
On Tue, Jun 23, 2015 at 9:22 AM, Francisco Jerez wrote: > Jason Ekstrand writes: > >> We want to move these into the builder so that they know the current >> builder's dispatch width. This will be needed by a later commit. > > I very much like the idea of this series, but, why do you need to mov

Re: [Mesa-dev] [PATCH 07/17] i965/fs: Move offset() and half() to the fs_builder

2015-06-23 Thread Francisco Jerez
Jason Ekstrand writes: > We want to move these into the builder so that they know the current > builder's dispatch width. This will be needed by a later commit. I very much like the idea of this series, but, why do you need to move these register manipulators into the builder? The builder is a

Re: [Mesa-dev] [PATCH 07/17] i965/fs: Move offset() and half() to the fs_builder

2015-06-23 Thread Iago Toral
On Thu, 2015-06-18 at 17:51 -0700, Jason Ekstrand wrote: > We want to move these into the builder so that they know the current > builder's dispatch width. This will be needed by a later commit. > --- > src/mesa/drivers/dri/i965/brw_fs.cpp | 52 ++ > src/mesa/drivers/dri/i965/brw

Re: [Mesa-dev] [PATCH 07/17] i965/fs: Move offset() and half() to the fs_builder

2015-06-23 Thread Pohjolainen, Topi
On Thu, Jun 18, 2015 at 05:51:36PM -0700, Jason Ekstrand wrote: > We want to move these into the builder so that they know the current > builder's dispatch width. This will be needed by a later commit. > --- > src/mesa/drivers/dri/i965/brw_fs.cpp | 52 ++ > src/mesa/drivers/dri/i

[Mesa-dev] [PATCH 07/17] i965/fs: Move offset() and half() to the fs_builder

2015-06-18 Thread Jason Ekstrand
We want to move these into the builder so that they know the current builder's dispatch width. This will be needed by a later commit. --- src/mesa/drivers/dri/i965/brw_fs.cpp | 52 ++ src/mesa/drivers/dri/i965/brw_fs_builder.h | 46 + src/mesa/drivers/dri/i965/brw_fs_c