On 07/07/2014 07:51 AM, Matthew McClure wrote:
With this patch, we add a test that will stress the bound resource limits
as determined by the GL API getInteger values. The bound resources are
populated with floats given a table of primes. Each contribution is
multiplied into the final expression,
On 07/07/2014 09:26 AM, Brian Paul wrote:
I'm having trouble with Mako since the requirement bump.
I had an older version of Mako installed. I tried upgrading with "sudo
pip install --user --upgrade Mako" but that didn't seem to work. Then I
uninstalled Mako and tried in
I'm having trouble with Mako since the requirement bump.
I had an older version of Mako installed. I tried upgrading with "sudo
pip install --user --upgrade Mako" but that didn't seem to work. Then I
uninstalled Mako and tried installing again. The installation had some
warnings but seems
ec/ext_framebuffer_multisample/accuracy.cpp
+++ b/tests/spec/ext_framebuffer_multisample/accuracy.cpp
@@ -58,7 +58,7 @@ const int pattern_width = 256; const int pattern_height = 256;
const int supersample_factor = 16;
Test *test = NULL;
-void
+NORETURN void
print_usage_and_exit(char *prog_name)
{
---
tests/spec/arb_provoking_vertex/xfb-before-flatshading.c |2 --
1 file changed, 2 deletions(-)
diff --git a/tests/spec/arb_provoking_vertex/xfb-before-flatshading.c
b/tests/spec/arb_provoking_vertex/xfb-before-flatshading.c
index 67cc581..6d168ea 100644
--- a/tests/spec/arb_provoking_ver
Since the test shaders require GLSL 1.30
---
.../arb_blend_func_extended/execution/fbo-extended-blend-explicit.c |2 +-
tests/spec/arb_blend_func_extended/execution/fbo-extended-blend.c|2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git
a/tests/spec/arb_blend_func_ext
On 06/30/2014 10:49 AM, Jason Ekstrand wrote:
On Sat, Jun 28, 2014 at 6:28 AM, Brian Paul mailto:bri...@vmware.com>> wrote:
On 06/27/2014 05:31 PM, Jason Ekstrand wrote:
This is a simple sanity test for the ARB_copy_image extension.
This tests
tes
On 06/27/2014 05:31 PM, Jason Ekstrand wrote:
This tests glCopyTexSubImage on all of the allowed combinations of internal
texture formats.
Signed-off-by: Jason Ekstrand
---
tests/all.py| 77 +++
tests/spec/arb_copy_image/CMakeLists.gl.txt | 1 +
tests/
On 06/27/2014 05:31 PM, Jason Ekstrand wrote:
This tests all of the possible different combinations of texture targets
such as GL_TEXTURE_2D, GL_TEXTURE_CUBE_MAP, GL_TEXTURE_1D_ARRAY, etc. The
only targets not tested are GL_TEXTURE_2D_MULTISAMPLE and
GL_TEXTURE_2D_MULTISAMPLE_ARRAY but those are
not sure if you want to do that here
though.
+
+ if (!piglit_automatic) {
+ glBindFramebufferEXT(GL_DRAW_FRAMEBUFFER_EXT,
+piglit_winsys_fbo);
+
+ image_draw(&images[1], 0, 0);
+ image_draw(&images[0], 64, 0);
+
+ piglit_present_results();
+ }
+
+ image_destroy(&images[0]);
+ image_destroy(&images[1]);
+
+ return pass ? PIGLIT_PASS : PIGLIT_FAIL;
+}
Looks good otherwise.
Reviewed-by: Brian Paul
___
Piglit mailing list
Piglit@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/piglit
This is based on the previous integer-errors test, but tests
many more glTexImage internalFormat / format / type combinations.
---
tests/all.py |1 +
tests/spec/ext_texture_integer/CMakeLists.gl.txt |1 +
tests/spec/ext_texture_integer/errors.c
Instead of spot checking a few internalFormat/format/type combinations
test all of them. And only test the core profile formats. The
ext_texture_integer-errors test hits the other legacy/compat formats.
Remove the tests for glDrawPixels, glReadPixels, etc. since that's
covered in the new ext_tex
For all 6 unused-var-removal patches:
Reviewed-by: Brian Paul
___
Piglit mailing list
Piglit@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/piglit
Correct.
-Brian
On 06/25/2014 04:53 PM, Dylan Baker wrote:
Wait, you guys don't have a 2.6 requirement anymore?
On Wednesday, June 25, 2014 04:48:11 PM Brian Paul wrote:
On 06/24/2014 07:24 PM, Kenneth Graunke wrote:
On Tuesday, June 24, 2014 10:30:56 AM Matt Turner wrote:
It was more
n.cpp | 96 ++--
1 file changed, 53 insertions(+), 43 deletions(-)
Reviewed-by: Brian Paul
___
Piglit mailing list
Piglit@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/piglit
On 06/24/2014 07:24 PM, Kenneth Graunke wrote:
On Tuesday, June 24, 2014 10:30:56 AM Matt Turner wrote:
It was more than a year ago now [1] that we agreed to "hold these
patches back for a short time (maybe a month) to give people time to
get ready for the transition"
In the mean time, we've st
GL_MAX_UNIFORM_BLOCK_SIZE is in bytes. A "component" is regarded as being
four bytes in size. So to compute the number of components, divide the
block size by 4.
With this change, nvidia's driver passes the test.
Found by Kai Ninomiya of VMware.
---
tests/spec/arb_uniform_buffer_object/minmax.
Even for functions such as glGetStringi() which are in OpenGL 3.0, we
call get_core_proc_address() with gl_10x_version = 10 (seems to be a
larger problem in the dispatch code generation). That means we try
GetProcAddress("OPENGL32.DLL") and fail. Now, if GetProcAddress()
fails, try get_ext_proc_a
Check if actually got a core profile. Then, if the tests requires
a compat profile, skip the test.
---
.../piglit-framework-gl/piglit_glut_framework.c| 22
1 file changed, 22 insertions(+)
diff --git a/tests/util/piglit-framework-gl/piglit_glut_framework.c
b/tests/uti
Thanks, but I think I'm going to do a bit more in this area. New
patch(es) to come, after more testing...
-Brian
On 06/25/2014 10:28 AM, Charmaine Lee wrote:
Reviewed-by: Charmaine Lee
From: Piglit on behalf of Brian Paul
Sent: Wednesday,
For the GLUT framework, check the context version and whether
GL_ARB_compatibility is supported to determine if we should set
piglit_is_core_profile. Fixes some issues when running GL 3.1 and
later on Windows.
---
tests/util/piglit-framework-gl/piglit_glut_framework.c |8
1 file chan
---
tests/spec/arb_texture_buffer_object/get.c |6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/tests/spec/arb_texture_buffer_object/get.c
b/tests/spec/arb_texture_buffer_object/get.c
index 202a909..d96a3c1 100644
--- a/tests/spec/arb_texture_buffer_object/get.c
+++ b/
ES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
+ * DEALINGS IN THE SOFTWARE.
+ */
+
+/**
+ * Test OpenGL 3.0 rendering to integer texture formats.
+ * Brian Paul
+ * 19 June 2014
This test requires a 2x2 window size- it's not just the glReadPixels(2x2)
that matters.
---
tests/general/pbo-readpixels-small.c |9 +
1 file changed, 9 insertions(+)
diff --git a/tests/general/pbo-readpixels-small.c
b/tests/general/pbo-readpixels-small.c
index 900a929..9f2f338 10064
en I (quickly) looked at this I assumed it was the
glReadPixels(w=2,h=2) that was significant. I'll post a fix-up patch...
-Brian
On 06/08/2014 03:16 PM, Brian Paul wrote:
---
tests/general/pbo-readpixels-small.c |3 ---
1 file changed, 3 deletions(-)
diff --git a/tests/general
stuff into the function too,
but no big deal.
Reviewed-by: Brian Paul
___
Piglit mailing list
Piglit@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/piglit
}
} else {
/* For stencil, there's no query for
* the type, so our table above
It might be nice to put the glGetTexLevelParameteriv() call and
surrounding checks into a separate fun
swizzles from red to green instead of to alpha.
Reviewed-by: Brian Paul
---
tests/all.py | 1 +
tests/fbo/CMakeLists.gl.txt| 1 +
tests/fbo/fbo-generatemipmap-swizzle.c | 165 +
3 files changed, 167 insertions
---
tests/spec/gl-3.0/vertexattribipointer.c | 43 +++---
1 file changed, 34 insertions(+), 9 deletions(-)
diff --git a/tests/spec/gl-3.0/vertexattribipointer.c
b/tests/spec/gl-3.0/vertexattribipointer.c
index b69e731..633b881 100644
--- a/tests/spec/gl-3.0/vertexattrib
piglit_require_extension("GL_ARB_texture_swizzle");
If you decide to keep the GL_RED texture, require GL_ARB_texture_rg.
+}
Looks good otherwise.
Reviewed-by: Brian Paul
___
Piglit mailing list
Piglit@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/piglit
Reviewed-by: Brian Paul
On Fri, Jun 13, 2014 at 11:03 PM, Vinson Lee wrote:
> Fix clang unused-const-variable warnings.
>
> tests/fbo/fbo-generatemipmap-1d.c:44:20: warning: unused variable 'red'
> [-Wunused-const-variable]
> static const fl
Ping.
On 06/11/2014 10:48 AM, Brian Paul wrote:
Add a test that actually draws something with uniform buffer objects
along the lines of what an application might do.
---
tests/all.py |1 +
.../arb_uniform_buffer_object/CMakeLists.gl.txt|1
Just to be clear, I think forward slash is / while backward slash is \.
-Brian
On 06/12/2014 06:22 AM, jfons...@vmware.com wrote:
From: José Fonseca
It's fine for URLs to have forward slash directory separators when viewing
the HTML on Windows. But if one moves the HTML report to a Linux mac
Add a test that actually draws something with uniform buffer objects
along the lines of what an application might do.
---
tests/all.py |1 +
.../arb_uniform_buffer_object/CMakeLists.gl.txt|1 +
tests/spec/arb_uniform_buffer_object/rendering.c |
)
+ext_packed_depth_stencil['readdrawpixels'] =
concurrent_test('ext_packed_depth_stencil-readdrawpixels')
oes_packed_depth_stencil = {}
spec['OES_packed_depth_stencil'] = oes_packed_depth_stencil
Reviewed-by: Brian Paul
Hmm, I did that but somehow it di
x27;: 'skip'}\n"
"Test executable not found.\n")
err = ""
-returncode = None
+returncode = 0
else:
raise e
Reviewed-by: Brian Paul
___
Pig
With nvidia's 325.15 driver (at least) the GLSL compiler generates invalid
code.
---
tests/all.py |8
.../execution/and-clamp.shader_test| 43
2 files changed, 51 insertions(+)
create mode 100644
tests/spec/arb
ceptionCode);
return EXCEPTION_CONTINUE_SEARCH;
}
The series looks OK to me, AFAICT.
Reviewed-by: Brian Paul
___
Piglit mailing list
Piglit@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/piglit
FTWARE OR THE USE OR OTHER DEALINGS
+ * IN THE SOFTWARE.
+ */
+
+/**
+ * Test GL_EXT_packed_depth_stencil with glRead/DrawPixels
+ * Based on an original Glean test written by Brian Paul.
+ */
+
+#include "piglit-util-gl-common.h"
+
+PIGLIT_GL_TEST_CONFIG_BEGIN
+ config.s
r API error handling.
+ * Based on an original Glean test written by Brian Paul.
+ */
+
+#include "piglit-util-gl-common.h"
+
+PIGLIT_GL_TEST_CONFIG_BEGIN
+ config.supports_gl_compat_version = 12;
+ config.window_visual = (PIGLIT_GL_VISUAL_RGBA |
+
R OTHER
-// DEALINGS IN THE SOFTWARE.
-//
-// END_COPYRIGHT
-
-// tbufferobject.c - test various buffer object features/extensions
-// Author: Brian Paul
-
-
-#define GL_GLEXT_PROTOTYPES
-#include
-#include
-#include
-#include
-#include "tbufferobject.h"
-
-
-namespace
-// DEALINGS IN THE SOFTWARE.
-//
-// END_COPYRIGHT
-
-// tdepthstencil.h: Test GL_EXT_packed_depth_stencil extension.
-// Brian Paul 1 October 2005
-
-
-#include
-#include
-#include
-#include "tdepthstencil.h"
-#include "rand.h"
-#include "image.h"
-#in
Use the larger of the default window size and the size required by
the test parameters.
---
tests/texturing/depth-tex-modes-rg.c |8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/tests/texturing/depth-tex-modes-rg.c
b/tests/texturing/depth-tex-modes-rg.c
index 4cf8aec
---
tests/shaders/fp-abs-01.c |3 ---
1 file changed, 3 deletions(-)
diff --git a/tests/shaders/fp-abs-01.c b/tests/shaders/fp-abs-01.c
index 0b59829..411c29b 100644
--- a/tests/shaders/fp-abs-01.c
+++ b/tests/shaders/fp-abs-01.c
@@ -37,9 +37,6 @@
PIGLIT_GL_TEST_CONFIG_BEGIN
config
---
tests/general/pbo-readpixels-small.c |3 ---
1 file changed, 3 deletions(-)
diff --git a/tests/general/pbo-readpixels-small.c
b/tests/general/pbo-readpixels-small.c
index 026531b..900a929 100644
--- a/tests/general/pbo-readpixels-small.c
+++ b/tests/general/pbo-readpixels-small.c
@@ -41,
---
tests/texturing/tex-skipped-unit.c |3 ---
1 file changed, 3 deletions(-)
diff --git a/tests/texturing/tex-skipped-unit.c
b/tests/texturing/tex-skipped-unit.c
index 53477d1..8355f41 100644
--- a/tests/texturing/tex-skipped-unit.c
+++ b/tests/texturing/tex-skipped-unit.c
@@ -33,9 +33,6 @@
then rendered to
the window and probed for the correct results.
---
Overall, the series looks good to me. Just a few nits that I posted.
Reviewed-by: Brian Paul
___
Piglit mailing list
Piglit@lists.freedesktop.org
http://lists.freedesktop.org
On 06/05/2014 09:38 AM, Neil Roberts wrote:
This adds a test for glClearTexSubImage when clearing a sub-region of
textures using an internal format of GL_RED and GL_RG.
---
tests/all.py | 1 +
tests/spec/arb_clear_texture/CMakeLists.gl.txt | 1 +
tests/spec
On 06/05/2014 09:38 AM, Neil Roberts wrote:
This adds a test for glClearTexSubImage when clearing a sub-region of
a texture using a range of sized internal formats.
---
tests/all.py | 1 +
tests/spec/arb_clear_texture/CMakeLists.gl.txt | 1 +
tests/spec/arb
On 06/05/2014 09:38 AM, Neil Roberts wrote:
This adds a test for glClearTexSubImage when clearing a sub-region of
a 2D texture using a range of formats that are supported in basic GL.
A utility function is added to test a format combination so that
further tests can be added with other formats th
LGTM. Reviewed-by: Brian Paul
On 06/05/2014 05:57 AM, Neil Roberts wrote:
This adds a test for updating a sub-region of a texture created with
the GL_EXT_packed_depth_stencil extension. Currently this seems to
trigger a bug on the i965 driver.
---
tests/all.py
n_handler);
#endif /* _WIN32 */
}
Looks good AFAICT.
Reviewed-by: Brian Paul
___
Piglit mailing list
Piglit@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/piglit
On 06/05/2014 05:51 AM, Jose Fonseca wrote:
Looks ok. Though I still wonder if it wouldn't be to do as I suggested in another
similar review request -- to treat config.window_width/height as "minimum"
values, as opposed to maximum values. That is, something along the lines of
diff --git a/te
I just did a quick review (you might wait to see if there's other
comments) but these tests look OK to me. Just a few minor things below.
Reviewed-by: Brian Paul
On 06/04/2014 11:13 AM, Neil Roberts wrote:
This adds a test for glClearTexSubImage when clearing a sub-region of
a 3D te
---
tests/all.py |1 +
tests/spec/gl-3.0/CMakeLists.gl.txt |1 +
tests/spec/gl-3.0/vertexattribipointer.c | 239 ++
3 files changed, 241 insertions(+)
create mode 100644 tests/spec/gl-3.0/vertexattribipointer.c
diff --git a/te
These tests need no other changes. Again, this is to avoid the "spurious
window resize" issue on Windows when the initial window size is too small.
---
tests/spec/ext_transform_feedback/alignment.c |3 ---
tests/spec/ext_transform_feedback/api-errors.c |3 ---
tests/spe
This test has a hard-coded array of expected results which assume
a 64 x 32 window. Now the window size can be anything but we
explicitly set up the projection matrix for a 64 x 32 region.
---
tests/spec/ext_transform_feedback/interleaved.c | 15 +++
1 file changed, 11 insertions(+)
This test also has hard-coded expected[] coords which assume a 64 x 32
window size. New use the default window size but setup the projection
matrix and viewport as if the window was only 64 x 32.
---
tests/spec/ext_transform_feedback/position.c | 16
1 file changed, 12 insertio
# Drop .py extension (used on Windows)
+_bin = _binname
_libdir = path.abspath(path.join(_binpath, '..', 'lib', _bin))
sys.path.append(_libdir)
if 'PIGLIT_SOURCE_DIR' not in os.environ:
For both: Reviewed-by: Brian Paul
---
tests/all.py | 11 +--
1 file changed, 5 insertions(+), 6 deletions(-)
diff --git a/tests/all.py b/tests/all.py
index e2d1c77..b8a246b 100644
--- a/tests/all.py
+++ b/tests/all.py
@@ -5,7 +5,6 @@ __all__ = ['profile']
import itertools
import os
-import os.path as path
import pl
---
tests/all.py |5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/tests/all.py b/tests/all.py
index 5802227..e2d1c77 100644
--- a/tests/all.py
+++ b/tests/all.py
@@ -27,9 +27,8 @@ if sys.platform == "cygwin":
# $PIGLIT_BUILD_DIR (if that environment variable exists) or
Use os.path.join() to build up file paths instead of using string
concatenation. In plain_test(), split up the arguments string
using string.split() instead of shlex.split() because the later
removes the '\' characters in Windows paths, which causes things to
blow up.
I can't find any particular
Otherwise, the file path we're passing to programs such as shader_runner
is in Unix style, which Windows programs don't understand.
v2: use subprocess module instead of commands module, per Dylan
---
tests/all.py |7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/tests/a
On 06/02/2014 11:14 AM, Jose Fonseca wrote:
- Original Message -
On Sunday, June 01, 2014 03:03:22 PM Brian Paul wrote:
Use os.path.join() to build up file paths instead of using string
concatenation. In plain_test(), split up the arguments string
using string.split() instead of
Tests all the type/size/normalize combinations for glVertexAttribPointer.
Note- a similar test should be written for GL 3.0's glVertexAttribIPointer.
v2: pass the expected value and atolerance to the shader and compute
pass/fail there, per Jose's idea. And test GL_DOUBLE type.
---
tests/all.py
On 06/02/2014 11:26 AM, Jose Fonseca wrote:
- Original Message -
Tests all the type/size/normalize combinations for glVertexAttribPointer.
Note- a similar test should be written for GL 3.0's glVertexAttribIPointer.
---
tests/all.py|1 +
tests/spec/gl-
On 06/01/2014 03:08 PM, Ilia Mirkin wrote:
On Sun, Jun 1, 2014 at 5:03 PM, Brian Paul wrote:
---
tests/asmparsertest/asmparsertest.c |1 +
1 file changed, 1 insertion(+)
diff --git a/tests/asmparsertest/asmparsertest.c
b/tests/asmparsertest/asmparsertest.c
index 16ed0df..c250270
---
tests/asmparsertest/asmparsertest.c |1 +
1 file changed, 1 insertion(+)
diff --git a/tests/asmparsertest/asmparsertest.c
b/tests/asmparsertest/asmparsertest.c
index 16ed0df..c250270 100644
--- a/tests/asmparsertest/asmparsertest.c
+++ b/tests/asmparsertest/asmparsertest.c
@@ -120,6 +120
Otherwise, the file path we're passing to programs such as shader_runner
is in Unix style, which Windows programs don't understand.
---
tests/all.py |5 +
1 file changed, 5 insertions(+)
diff --git a/tests/all.py b/tests/all.py
index a42e79e..bf174b7 100644
--- a/tests/all.py
+++ b/tests/
Use os.path.join() to build up file paths instead of using string
concatenation. In plain_test(), split up the arguments string
using string.split() instead of shlex.split() because the later
removes the '\' characters in Windows paths, which causes things to
blow up.
I can't find any particular
Reviewed-by: Brian Paul
On Sun, Jun 1, 2014 at 1:54 PM, wrote:
> From: José Fonseca
>
> Mesa didn't get this quite right, causing problems to ApiTrace --
> https://github.com/apitrace/apitrace/issues/261 .
> ---
> tests/all.py
Tests all the type/size/normalize combinations for glVertexAttribPointer.
Note- a similar test should be written for GL 3.0's glVertexAttribIPointer.
---
tests/all.py|1 +
tests/spec/gl-2.0/CMakeLists.gl.txt |1 +
tests/spec/gl-2.0/vertexattribpointer.c | 2
On Fri, May 30, 2014 at 10:48 AM, Jose Fonseca wrote:
>
> Series looks good at a glance. Just one minor remark inline.
>
>
> > + */
> > +
> > +
> > +#ifndef PIGLIT_MATRIX_H
> > +#define PIGLIT_MATRIX_H
> > +
> > +
> > +#ifdef __cplusplus
> > +extern "C" {
> > +#endif
> > +
> > +
> > +#include
>
---
tests/spec/glsl-1.30/execution/vertexid-beginend.c |3 ---
tests/spec/glsl-1.30/execution/vertexid-drawarrays.c |3 ---
tests/spec/glsl-1.30/execution/vertexid-drawelements.c |3 ---
3 files changed, 9 deletions(-)
diff --git a/tests/spec/glsl-1.30/execution/vertexid-beginen
On 05/24/2014 05:31 PM, Emil Velikov wrote:
On 24/05/14 22:10, Ilia Mirkin wrote:
Signed-off-by: Ilia Mirkin
---
This doesn't actually end up mattering for the GLX tests because they do a
substr-based search in the overall extension string. But for the GL tests it
matters because it looks for
(arg0);
gl_FragColor = result == expected ? vec4(0.0, 1.0, 0.0, 1.0) :
vec4(1.0, 0.0, 0.0, 1.0);
}
On Fri, May 23, 2014 at 1:18 PM, Brian Paul wrote:
---
.../glsl-1.30/execution/fs-sign-int4.shader_test | 24 +
.../glsl-1.30/execution/vs-sign-int4.shader_test |
---
.../glsl-1.30/execution/fs-sign-int4.shader_test | 24 +
.../glsl-1.30/execution/vs-sign-int4.shader_test | 27
2 files changed, 51 insertions(+)
create mode 100644 tests/spec/glsl-1.30/execution/fs-sign-int4.shader_test
create mode 100644 tests/s
This test uses the new piglit matrix functions to setup
transformations.
---
tests/all.py |1 +
tests/spec/arb_base_instance/CMakeLists.gl.txt |1 +
tests/spec/arb_base_instance/drawarrays.c | 226
3 files changed, 228 insert
For tests using the core profile we no longer have the legacy
GL matrix functions to set up transformations. These functions
let us easily setup transformations in such test programs.
---
tests/util/CMakeLists.gl.txt |1 +
tests/util/piglit-matrix.c | 458 ++
On 05/21/2014 04:08 PM, Brian Paul wrote:
Please disregard.
-Brian
___
Piglit mailing list
Piglit@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/piglit
This enables the GL_ARB_instanced_arrays extension (vertex attribute
divisors). Piglit tests pass.
---
src/gallium/drivers/svga/svga_pipe_vertex.c | 10 --
src/gallium/drivers/svga/svga_screen.c |2 +-
2 files changed, 9 insertions(+), 3 deletions(-)
diff --git a/src/gallium/d
On 05/20/2014 04:31 PM, Matt Turner wrote:
From: Kenneth Graunke
---
tests/glean/tglsl1.cpp | 152 +
tests/shaders/glsl1/COPYRIGHT | 31 +
tests/shaders/glsl1/break-with-no-loop.frag| 9 ++
tests/shaders/gl
Prefix the executables with "arb_instanced_arrays" to be more
consistent.
---
tests/all.py |6 +++---
tests/spec/arb_instanced_arrays/CMakeLists.gl.txt |4 ++--
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/tests/all.py b/tests/all.py
i
---
tests/all.py |2 +-
.../CMakeLists.gl.txt |2 +-
.../dlist-arb_draw_instanced.c | 96
tests/spec/arb_draw_elements_base_vertex/dlist.c | 96
4 files ch
---
tests/all.py |4 +-
tests/general/CMakeLists.gl.txt|4 -
tests/general/draw-elements-base-vertex-neg.c | 141
.../CMakeLists.gl.txt |1 +
.../arb_draw_elements_base_ver
---
tests/all.py |2 +-
.../CMakeLists.gl.txt |2 +-
.../multi-draw-elements-base-vertex.c | 190
.../multidrawelements.c| 190
4 files ch
---
tests/all.py |2 +-
tests/general/CMakeLists.gl.txt|1 -
tests/general/draw-elements-base-vertex-bounds.c | 127
.../CMakeLists.gl.txt |1 +
tests/spec/arb_draw_elements_b
---
tests/all.py |2 +-
.../CMakeLists.gl.txt |2 +-
.../draw-elements-instanced-base-vertex.c | 191
.../drawelements-instanced.c | 191
4 files ch
The names of many piglit tests are pretty inconsistent.
Nowadays, most tests get put in a tests/spec/arb_foo_bar/ directory and
the executables are prefixed with "arb_foo_bar-*". But older tests
don't really follow any pattern. Many were just dumped in tests/general/
I'd like to clean up s
---
tests/all.py |2 +-
.../CMakeLists.gl.txt |2 +-
.../draw-range-elements-base-vertex.c | 173
.../drawrangeelements.c| 173
4 files ch
---
tests/all.py |5 +-
tests/general/CMakeLists.gl.txt|1 -
tests/general/draw-elements-base-vertex.c | 156
.../CMakeLists.gl.txt |1 +
.../arb_draw_elements_base_ver
Move tests/general/draw-instanced.c to
tests/spec/arb_draw_instanced/drawarrays.c
---
tests/all.py |2 +-
tests/general/CMakeLists.gl.txt|1 -
tests/general/draw-instanced.c | 183
.../arb_d
Move tests/general/draw-instanced-divisor.c to
tests/spec/arb_instanced_arrays/drawarrays.c
---
tests/all.py |2 +-
tests/general/CMakeLists.gl.txt |1 -
tests/general/draw-instanced-divisor.c| 191 -
t
---
tests/spec/arb_draw_instanced/execution/elements.c |5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/tests/spec/arb_draw_instanced/execution/elements.c
b/tests/spec/arb_draw_instanced/execution/elements.c
index 7cb418b..84eed79 100644
--- a/tests/spec/arb_draw_instan
---
.../arb_texture_compression/internal-format-query.c | 17 +
1 file changed, 9 insertions(+), 8 deletions(-)
diff --git a/tests/spec/arb_texture_compression/internal-format-query.c
b/tests/spec/arb_texture_compression/internal-format-query.c
index 7a7b6d4..f6b2af2 100644
-
Commit ef4f461346d01a66 mistakenly removed the window size from this
test, causing it to fail.
---
tests/spec/arb_occlusion_query/occlusion_query.c |3 +++
1 file changed, 3 insertions(+)
diff --git a/tests/spec/arb_occlusion_query/occlusion_query.c
b/tests/spec/arb_occlusion_query/occlusion
On 05/14/2014 09:53 PM, Michel Dänzer wrote:
On 14.05.2014 10:06, Brian Paul wrote:
diff --git a/tests/spec/arb_occlusion_query/occlusion_query.c
b/tests/spec/arb_occlusion_query/occlusion_query.c
index e099f06..b8ea664 100644
--- a/tests/spec/arb_occlusion_query/occlusion_query.c
+++ b/tests
On 05/14/2014 12:31 PM, Dylan Baker wrote:
On Wednesday, May 14, 2014 12:23:04 Brian Paul wrote:
> Dylan, I think you may have forgot to commit a new file or something:
>
> Traceback (most recent call last):
> File "/home/brianp/pig/generated_tests/gen_texture_lod_tests.
Dylan, I think you may have forgot to commit a new file or something:
Traceback (most recent call last):
File "/home/brianp/pig/generated_tests/gen_texture_lod_tests.py",
line 178, in
main()
File "/home/brianp/pig/generated_tests/gen_texture_lod_tests.py",
line 160, in main
gen_f
Yup, that'd be easier for me too. I've got more of these changes
coming, but now I'm down to the ones which require fixing the test
because they relied on particular window dimensions.
-Brian
On 05/14/2014 08:46 AM, Jose Fonseca wrote:
LGTM AFAICT.
BTW, personally, I'd prefer reviewing this
701 - 800 of 1411 matches
Mail list logo
