Re: [Mesa-dev] [PATCH 000/123] Reintroducing NIR, a new IR for mesa

2015-01-14 Thread Connor Abbott
On Wed, Jan 14, 2015 at 6:53 PM, Jason Ekstrand wrote: > > > On Sun, Jan 11, 2015 at 7:52 PM, Connor Abbott wrote: >> >> Patches without my Reviewed-by on them on your branch are: >> >> "i965/fs_nir: Add support for sample_pos and sample_id" >> >> This one gets rewritten later on so I think it's

Re: [Mesa-dev] [PATCH 000/123] Reintroducing NIR, a new IR for mesa

2015-01-14 Thread Jason Ekstrand
On Sun, Jan 11, 2015 at 7:52 PM, Connor Abbott wrote: > Patches without my Reviewed-by on them on your branch are: > > "i965/fs_nir: Add support for sample_pos and sample_id" > > This one gets rewritten later on so I think it's ok to not review it. > > "i965/fs: Allow reinterpretation in constant

Re: [Mesa-dev] [PATCH 000/123] Reintroducing NIR, a new IR for mesa

2015-01-11 Thread Connor Abbott
Patches without my Reviewed-by on them on your branch are: "i965/fs_nir: Add support for sample_pos and sample_id" This one gets rewritten later on so I think it's ok to not review it. "i965/fs: Allow reinterpretation in constant propagation" We need Matt or Ken to review this. "i965/fs: Don't

Re: [Mesa-dev] [PATCH 000/123] Reintroducing NIR, a new IR for mesa

2015-01-06 Thread Jason Ekstrand
On Mon, Jan 5, 2015 at 11:28 PM, Connor Abbott wrote: > Patches 100, 102-104, 106-109, 114-117, 121-146 are: > > Reviewed-by: Connor Abbott > Noted. I'm working on dealing with comments now. --Jason > > Whew! I think I've looked through everything. > > On Tue, Dec 16, 2014 at 1:04 AM, Jason

Re: [Mesa-dev] [PATCH 000/123] Reintroducing NIR, a new IR for mesa

2015-01-05 Thread Connor Abbott
Patches 100, 102-104, 106-109, 114-117, 121-146 are: Reviewed-by: Connor Abbott Whew! I think I've looked through everything. On Tue, Dec 16, 2014 at 1:04 AM, Jason Ekstrand wrote: > NIR (pronounced "ner") is a new IR (internal representation) for the Mesa > shader compiler that will sit betw

Re: [Mesa-dev] [PATCH 000/123] Reintroducing NIR, a new IR for mesa

2014-12-17 Thread Connor Abbott
On Wed, Dec 17, 2014 at 1:59 AM, Connor Abbott wrote: > Patches 23-26, 28, 30-35, 37-38, 40, (41 gets killed later so I didn't > review it), 42-44 are > > Reviewed-by: Connor Abbott > > I'm going to bed now, but I'll try to do some more later. Patches 47-48, 52-54, 56-58, 60-64, 66-70, 72-73, 76

Re: [Mesa-dev] [PATCH 000/123] Reintroducing NIR, a new IR for mesa

2014-12-17 Thread Jason Ekstrand
For my own sake as well of those reviewing things, I'm keeping both the review/nir-v1 branch and http://patchwork.freedesktop.org/bundle/jekstrand/nir-v1/ up-to-date as new patches get sent out and reviewed-by's get added. On Tue, Dec 16, 2014 at 10:59 PM, Connor Abbott wrote: > > Patches 23-26,

Re: [Mesa-dev] [PATCH 000/123] Reintroducing NIR, a new IR for mesa

2014-12-16 Thread Connor Abbott
Patches 23-26, 28, 30-35, 37-38, 40, (41 gets killed later so I didn't review it), 42-44 are Reviewed-by: Connor Abbott I'm going to bed now, but I'll try to do some more later. On Tue, Dec 16, 2014 at 1:04 AM, Jason Ekstrand wrote: > NIR (pronounced "ner") is a new IR (internal representation

Re: [Mesa-dev] [PATCH 000/123] Reintroducing NIR, a new IR for mesa

2014-12-16 Thread Connor Abbott
Oh, I forgot... patch 20 should probably get cherry-picked as well. On Tue, Dec 16, 2014 at 6:01 PM, Jason Ekstrand wrote: > > > On Tue, Dec 16, 2014 at 2:52 PM, Connor Abbott wrote: >> >> Hi, >> >> On Tue, Dec 16, 2014 at 1:04 AM, Jason Ekstrand >> wrote: >> > NIR (pronounced "ner") is a new I

Re: [Mesa-dev] [PATCH 000/123] Reintroducing NIR, a new IR for mesa

2014-12-16 Thread Jason Ekstrand
On Tue, Dec 16, 2014 at 2:52 PM, Connor Abbott wrote: > > Hi, > > On Tue, Dec 16, 2014 at 1:04 AM, Jason Ekstrand > wrote: > > NIR (pronounced "ner") is a new IR (internal representation) for the Mesa > > shader compiler that will sit between the old IR (GLSL IR) and back-end > > compilers. The

Re: [Mesa-dev] [PATCH 000/123] Reintroducing NIR, a new IR for mesa

2014-12-16 Thread Connor Abbott
Hi, On Tue, Dec 16, 2014 at 1:04 AM, Jason Ekstrand wrote: > NIR (pronounced "ner") is a new IR (internal representation) for the Mesa > shader compiler that will sit between the old IR (GLSL IR) and back-end > compilers. The primary purpose of NIR is to be more efficient for doing > optimizatio

[Mesa-dev] [PATCH 000/123] Reintroducing NIR, a new IR for mesa

2014-12-15 Thread Jason Ekstrand
NIR (pronounced "ner") is a new IR (internal representation) for the Mesa shader compiler that will sit between the old IR (GLSL IR) and back-end compilers. The primary purpose of NIR is to be more efficient for doing optimizations and generate better code for the back-ends. We have a lot of opti