[Mesa-dev] [PATCH V3 3/6] meta: Add functionality to do _mesa_meta_BlitFrameBuffer() using glsl

2013-01-01 Thread Anuj Phogat
This patch rewrites _mesa_meta_BlitFrameBuffer() function to add support for blitting with GLSL/GLSL ES shaders. These changes were required to support glBlitFrameBuffer() in gles3. This patch, along with other patches in this series, make 16 failing framebuffer_blit test cases in gles3 conformance

Re: [Mesa-dev] [PATCH] i965: Add break statement at end of BRW_OPCODE_CONTINUE case.

2013-01-01 Thread Kenneth Graunke
On 01/01/2013 05:06 PM, Vinson Lee wrote: Fixes missing break in switch defect reported by Coverity. Signed-off-by: Vinson Lee --- src/mesa/drivers/dri/i965/brw_eu_emit.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c b/src/mesa/drivers/dri/i965/

Re: [Mesa-dev] [PATCH] i965: Add break statement at end of BRW_OPCODE_CONTINUE case.

2013-01-01 Thread Paul Berry
This looks correct to me. Reviewed-by: Paul Berry On 1 January 2013 17:06, Vinson Lee wrote: > Fixes missing break in switch defect reported by Coverity. > > Signed-off-by: Vinson Lee > --- > src/mesa/drivers/dri/i965/brw_eu_emit.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/sr

[Mesa-dev] [PATCH] i965: Add break statement at end of BRW_OPCODE_CONTINUE case.

2013-01-01 Thread Vinson Lee
Fixes missing break in switch defect reported by Coverity. Signed-off-by: Vinson Lee --- src/mesa/drivers/dri/i965/brw_eu_emit.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c b/src/mesa/drivers/dri/i965/brw_eu_emit.c index 8a93ced..bd9c182 100644 --

Re: [Mesa-dev] [PATCH] configure.ac: Remove space after indent -T flag.

2013-01-01 Thread Vinson Lee
On Mon, Dec 31, 2012 at 5:31 PM, Matt Turner wrote: > On Mon, Dec 31, 2012 at 3:27 PM, Vinson Lee wrote: >> Fixes this build error on platforms not using GNU indent. >> >> indent: Command line: ``-T'' requires a parameter >> >> Signed-off-by: Vinson Lee >> --- >> configure.ac | 2 +- >> 1 file

[Mesa-dev] [PATCH] i965/fs: Remove force_sechalf stack.

2013-01-01 Thread Kenneth Graunke
Only Gen4 color write setup uses the force_sechalf flag, and it only sets it on a single instruction. It also already has to get a pointer to the instruction and manually set the saturate flag, so we may as well just set force_sechalf the same way and avoid the complexity of a stack. --- src/mesa

Re: [Mesa-dev] When is a sampler object not a sampler object?

2013-01-01 Thread Kenneth Graunke
On 01/01/2013 09:20 AM, Matt Turner wrote: On Wed, Dec 19, 2012 at 8:40 PM, Matt Turner wrote: The ES 3.0 and GL 3.3+ specs say: A new sampler object is created by binding an unused name to a texture unit. [...] The names are marked as used, for the purposes of GenSamplers only,

Re: [Mesa-dev] When is a sampler object not a sampler object?

2013-01-01 Thread Matt Turner
On Wed, Dec 19, 2012 at 8:40 PM, Matt Turner wrote: > The ES 3.0 and GL 3.3+ specs say: > > A new sampler object is created by binding an unused name to a > texture unit. > > [...] > > The names are marked as used, for the purposes of GenSamplers only, > but they acquire state only when they are f