Re: [Mesa-dev] [PATCH] glsl: Eliminate unused built-in variables after compilation

2014-05-31 Thread Kenneth Graunke
On Thursday, May 29, 2014 02:37:34 PM Ian Romanick wrote: > From: Ian Romanick > > After compilation (and before linking) we can eliminate quite a few > built-in variables. Basically, any uniform or constant (e.g., > gl_MaxVertexTextureImageUnits) that isn't used (with one exception) can > be el

Re: [Mesa-dev] [PATCH 2/3] radeon/compute: Implement PIPE_COMPUTE_CAP_MAX_COMPUTE_UNITS

2014-05-31 Thread Bruno Jimenez
On Fri, 2014-05-30 at 19:33 -0400, Alex Deucher wrote: > On Fri, May 30, 2014 at 11:31 AM, Bruno Jiménez wrote: > > The data has been extracted from: > > AMD Accelerated Parallel Processing OpenCL Programming Guide (rev 2.7) > > Appendix D: Device Parameters > > You should add a query for the num

Re: [Mesa-dev] [PATCH 1/2] glsl: Validate that built-in uniforms have backing state

2014-05-31 Thread Kenneth Graunke
On Wednesday, May 28, 2014 06:35:47 PM Ian Romanick wrote: > From: Ian Romanick > > All built-in uniforms are supposed to be backed by some GL state. The > state_slots field describes this backing state. > > This helped me track down a bug in a later patch. > > Signed-off-by: Ian Romanick > -

[Mesa-dev] [Bug 79469] Commit e3cc0d90e14e62a0a787b6c07a6df0f5c84039be breaks unigine heaven

2014-05-31 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79469 David Heidelberger (okias) changed: What|Removed |Added CC||david.heidelber...@ixit.cz

[Mesa-dev] [Bug 79039] [TRACKER] Mesa 10.2 release tracker

2014-05-31 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79039 Bug 79039 depends on bug 79294, which changed state. Bug 79294 Summary: Xlib-based build broken on non x86/x86-64 architectures https://bugs.freedesktop.org/show_bug.cgi?id=79294 What|Removed |Added ---

[Mesa-dev] [Bug 79469] Commit e3cc0d90e14e62a0a787b6c07a6df0f5c84039be breaks unigine heaven

2014-05-31 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79469 Matt Turner changed: What|Removed |Added CC||a...@nwnk.net, |

[Mesa-dev] [Bug 79421] [llvmpipe] SIGSEGV src/gallium/drivers/llvmpipe/lp_rast_priv.h:218

2014-05-31 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79421 Roland Scheidegger changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Mesa-dev] [Bug 79421] [llvmpipe] SIGSEGV src/gallium/drivers/llvmpipe/lp_rast_priv.h:218

2014-05-31 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79421 --- Comment #2 from Roland Scheidegger --- I still have no idea why this could happen with the fb size used by this piglit test but fixed with 576868140bbb1abd177e7fd122720883d773137e. -- You are receiving this mail because: You are the assigne

Re: [Mesa-dev] [PATCH] mesa: Free the compiled shader IR after it has been linked.

2014-05-31 Thread Eric Anholt
Ian Romanick writes: > On 05/28/2014 01:57 PM, Eric Anholt wrote: >> If the shader compiled once, then we can compile it again. Compiled >> shaders almost always get used in just one program, so holding that >> compiled IR until the program is freed is just a waste of memory. > > Would this work

Re: [Mesa-dev] [PATCH 4/5] i965/fs: Debug the optimization passes by dumping instr to file.

2014-05-31 Thread Kenneth Graunke
On Friday, May 30, 2014 07:35:22 PM Matt Turner wrote: > With INTEL_DEBUG=optimizer, write the output of dump_instructions() to a > file each time an optimization pass makes progress. This lets you easily > diff successive files to see what an optimization pass did. > > Example filenames written w

Re: [Mesa-dev] [PATCH 4/5] i965/fs: Debug the optimization passes by dumping instr to file.

2014-05-31 Thread Matt Turner
On Sat, May 31, 2014 at 5:33 PM, Kenneth Graunke wrote: > On Friday, May 30, 2014 07:35:22 PM Matt Turner wrote: >> With INTEL_DEBUG=optimizer, write the output of dump_instructions() to a >> file each time an optimization pass makes progress. This lets you easily >> diff successive files to see w

Re: [Mesa-dev] [PATCH 5/5] i965: Skip IR annotations with INTEL_DEBUG=noann.

2014-05-31 Thread Kenneth Graunke
On Friday, May 30, 2014 07:35:23 PM Matt Turner wrote: > Running shader-db with INTEL_DEBUG=noann reduces the runtime > from ~90 to ~80 seconds on my machine. It also reduces the disk space > consumed by the .out files from 660 MB (676 on disk) to 343 MB (358 on > disk). > --- > src/mesa/drivers/d

Re: [Mesa-dev] [PATCH 5/5] i965: Skip IR annotations with INTEL_DEBUG=noann.

2014-05-31 Thread Matt Turner
On Sat, May 31, 2014 at 5:52 PM, Kenneth Graunke wrote: > On Friday, May 30, 2014 07:35:23 PM Matt Turner wrote: >> Running shader-db with INTEL_DEBUG=noann reduces the runtime >> from ~90 to ~80 seconds on my machine. It also reduces the disk space >> consumed by the .out files from 660 MB (676 o

Re: [Mesa-dev] [PATCH 19/19] i965/fs: Optimize SEL with the same sources into a MOV.

2014-05-31 Thread Kenneth Graunke
On Tuesday, May 27, 2014 06:47:50 PM Matt Turner wrote: > instructions in affected programs: 474 -> 462 (-2.53%) > --- > src/mesa/drivers/dri/i965/brw_fs.cpp | 7 ++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers

Re: [Mesa-dev] [PATCH 18/19] i965/fs: Perform CSE on texture operations.

2014-05-31 Thread Kenneth Graunke
On Tuesday, May 27, 2014 06:47:49 PM Matt Turner wrote: > Helps Unigine Tropics and some (old) gstreamer shaders in shader-db. > > instructions in affected programs: 792 -> 744 (-6.06%) > --- > src/mesa/drivers/dri/i965/brw_fs_cse.cpp | 12 +++- > 1 file changed, 11 insertions(+), 1 d

Re: [Mesa-dev] [PATCH] mesa: Free the compiled shader IR after it has been linked.

2014-05-31 Thread Ian Romanick
On 05/31/2014 02:39 PM, Eric Anholt wrote: > Ian Romanick writes: > >> On 05/28/2014 01:57 PM, Eric Anholt wrote: >>> If the shader compiled once, then we can compile it again. Compiled >>> shaders almost always get used in just one program, so holding that >>> compiled IR until the program is f

[Mesa-dev] [Bug 79469] Commit e3cc0d90e14e62a0a787b6c07a6df0f5c84039be breaks unigine heaven

2014-05-31 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79469 Ian Romanick changed: What|Removed |Added CC||i...@freedesktop.org --- Comment #4 from

[Mesa-dev] [Bug 79039] [TRACKER] Mesa 10.2 release tracker

2014-05-31 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79039 Ian Romanick changed: What|Removed |Added Depends on||79469 -- You are receiving this mail bec

[Mesa-dev] [Bug 79469] Commit e3cc0d90e14e62a0a787b6c07a6df0f5c84039be breaks unigine heaven

2014-05-31 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79469 Ian Romanick changed: What|Removed |Added Blocks||79039 -- You are receiving this mail bec

Re: [Mesa-dev] [PATCH] mesa: Free the compiled shader IR after it has been linked.

2014-05-31 Thread Ian Romanick
On 05/28/2014 01:57 PM, Eric Anholt wrote: > If the shader compiled once, then we can compile it again. Compiled > shaders almost always get used in just one program, so holding that > compiled IR until the program is freed is just a waste of memory. > > On the other hand, if they are either reus

Re: [Mesa-dev] [PATCH 00/21] Reduce ir_variable memory usage

2014-05-31 Thread Ian Romanick
On 05/30/2014 05:52 PM, Kenneth Graunke wrote: > On Friday, May 30, 2014 11:07:03 AM Eric Anholt wrote: >> Ian Romanick writes: >>> This series reduces the memory usage of ir_variable quite significantly. >>> >>> The first couple patches add a mechanism to determine the amount of >>> memory used b

Re: [Mesa-dev] [PATCH 04/19] i965/fs: Store the number of sources an fs_inst has.

2014-05-31 Thread Kenneth Graunke
On Tuesday, May 27, 2014 06:47:35 PM Matt Turner wrote: > --- > src/mesa/drivers/dri/i965/brw_fs.cpp | 21 +++-- > src/mesa/drivers/dri/i965/brw_fs.h | 3 ++- > 2 files changed, 13 insertions(+), 11 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/

Re: [Mesa-dev] [PATCH 05/19] i965/fs: Loop from 0 to inst->sources, not 0 to 3.

2014-05-31 Thread Kenneth Graunke
On Tuesday, May 27, 2014 06:47:36 PM Matt Turner wrote: [snip] > diff --git a/src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp b/src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp > index 069b60f..a1aff21 100644 > --- a/src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp > +++ b/src/mesa

Re: [Mesa-dev] [PATCH 10/19] i965/fs: Lower LOAD_PAYLOAD and clean up.

2014-05-31 Thread Kenneth Graunke
On Tuesday, May 27, 2014 06:47:41 PM Matt Turner wrote: > Clean up with with register_coalesce()/dead_code_eliminate(). > --- > src/mesa/drivers/dri/i965/brw_fs.cpp | 42 > src/mesa/drivers/dri/i965/brw_fs.h | 1 + > 2 files changed, 43 insertions(+) > > d

Re: [Mesa-dev] [PATCH 11/19] i965/fs: Use LOAD_PAYLOAD in emit_texture_gen7().

2014-05-31 Thread Kenneth Graunke
On Friday, April 18, 2014 11:56:47 AM Matt Turner wrote: > --- > src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 135 +++ > 1 file changed, 73 insertions(+), 62 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp b/src/mesa/drivers/dri/i965/brw_fs_v