Re: [Piglit] [PATCH 1/2] arb_gpu_shader_fp64: fix wrong use of GLSL version

2015-11-18 Thread Emil Velikov
On 17 November 2015 at 15:25, Samuel Pitoiset wrote: > piglit_require_GLSL_version() requires an integer as parameter, not a float. > This fixes an implicit conversion spotted by Clang. > > Signed-off-by: Samuel Pitoiset > --- >

Re: [Piglit] [PATCH 1/2] arb_gpu_shader_fp64: fix wrong use of GLSL version

2015-11-18 Thread Dave Airlie
On 19 November 2015 at 02:05, Emil Velikov wrote: > On 17 November 2015 at 15:25, Samuel Pitoiset > wrote: >> piglit_require_GLSL_version() requires an integer as parameter, not a float. >> This fixes an implicit conversion spotted by Clang.

[Piglit] [PATCH] arb_draw_indirect: test sharing binding point with 2 attribs

2015-11-18 Thread Tapani Pälli
Test passes with Nvidia binary driver and on current Mesa. Signed-off-by: Tapani Pälli --- tests/all.py | 1 + tests/spec/arb_draw_indirect/CMakeLists.gl.txt | 1 + .../arb_draw_indirect/draw-arrays-shared-binding.c | 133

Re: [Piglit] [PATCH] arb_separate_shader_objects: test mixed explicit and non-explicit locations

2015-11-18 Thread Gregory Hainaut
Hello Timothy, Normally the first of the 2 new SSO tests that I sent a couple of weeks ago cover this case. However my test is limited to the first location. Hopefully my Mesa patches take care of the issue. I will check it with this new test (and rebase my patches) Best regards, Gregory On

Re: [Piglit] [PATCH 1/2] SSO: new test to ensure correct deadcode optimization

2015-11-18 Thread Timothy Arceri
On Tue, 2015-11-03 at 20:09 +0100, Gregory Hainaut wrote: > Related to mesa issue: https://bugs.freedesktop.org/show_bug.cgi?id=79783 > > Mostly "Validated" on Nvidia driver. Nvidia fails to link > vs_fs_prog_separate_inactive It also fails to link on the i965 driver. Failed to link: error:

[Piglit] [PATCH 4/4] ext_shader_samples_identical: Simple fragment shader rendering test

2015-11-18 Thread Ian Romanick
From: Ian Romanick Render a simple image. Scan the image. At each texel, render blue if textureSamplesIdenticalEXT returns false. If textureSamplesIdenticalEXT returns true, examine each sample. If the samples are all the same color, render green. Render red

Re: [Piglit] [PATCH 1/4] arb_texture_multisample: Use piglit_vertex and piglit_texcoord to avoid glGetAttribLocation

2015-11-18 Thread Ilia Mirkin
On Wed, Nov 18, 2015 at 8:46 PM, Ian Romanick wrote: > diff --git a/tests/spec/CMakeLists.txt b/tests/spec/CMakeLists.txt > index a5c42dc..408649d 100644 > --- a/tests/spec/CMakeLists.txt > +++ b/tests/spec/CMakeLists.txt > @@ -76,6 +76,7 @@ add_subdirectory

Re: [Piglit] [PATCH 4/4] ext_shader_samples_identical: Simple fragment shader rendering test

2015-11-18 Thread Ian Romanick
On 11/18/2015 05:46 PM, Ian Romanick wrote: > From: Ian Romanick > > Render a simple image. Scan the image. At each texel, render blue if > textureSamplesIdenticalEXT returns false. If textureSamplesIdenticalEXT > returns true, examine each sample. If the samples

Re: [Piglit] [PATCH 1/2] SSO: new test to ensure correct deadcode optimization

2015-11-18 Thread Timothy Arceri
On Thu, 2015-11-19 at 12:14 +1100, Timothy Arceri wrote: > On Tue, 2015-11-03 at 20:09 +0100, Gregory Hainaut wrote: > > Related to mesa issue: https://bugs.freedesktop.org/show_bug.cgi?id=79783 > > > > Mostly "Validated" on Nvidia driver. Nvidia fails to link > > vs_fs_prog_separate_inactive >

[Piglit] [PATCH 1/4] arb_texture_multisample: Use piglit_vertex and piglit_texcoord to avoid glGetAttribLocation

2015-11-18 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- tests/spec/CMakeLists.txt | 1 + tests/spec/arb_texture_multisample/texelfetch.c | 34 +++-- 2 files changed, 16 insertions(+), 19 deletions(-)

[Piglit] [PATCH 2/4] ext_shader_samples_identical: add test for preprocessor define

2015-11-18 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- .../glsl-1.10/preprocessor/define.frag| 19 +++ .../glsl-1.10/preprocessor/define.vert| 19 +++

[Piglit] [PATCH 3/4] ext_shader_samples_identical: Verify that every function overload exists

2015-11-18 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- .../glsl-1.10/compiler/all-functions.frag | 21 + .../glsl-1.10/compiler/all-functions.vert | 21 + .../glsl-1.50/compiler/all-functions.frag

Re: [Piglit] [PATCH 1/4] arb_texture_multisample: Use piglit_vertex and piglit_texcoord to avoid glGetAttribLocation

2015-11-18 Thread Ian Romanick
On 11/18/2015 06:12 PM, Ilia Mirkin wrote: > On Wed, Nov 18, 2015 at 8:46 PM, Ian Romanick wrote: >> diff --git a/tests/spec/CMakeLists.txt b/tests/spec/CMakeLists.txt >> index a5c42dc..408649d 100644 >> --- a/tests/spec/CMakeLists.txt >> +++ b/tests/spec/CMakeLists.txt >>

Re: [Piglit] [PATCH 1/2] SSO: new test to ensure correct deadcode optimization

2015-11-18 Thread Timothy Arceri
On Tue, 2015-11-03 at 20:09 +0100, Gregory Hainaut wrote: > Related to mesa issue: https://bugs.freedesktop.org/show_bug.cgi?id=79783 > > Mostly "Validated" on Nvidia driver. Nvidia fails to link > vs_fs_prog_separate_inactive > > v3: add the test to all.py > > v2: > * Test both output and