Re: [Piglit] [PATCH] tests/spec/arb_robustness/draw-vbo-bounds.c: add clipping

2012-10-29 Thread Roland Scheidegger
Am 30.10.2012 00:22, schrieb Brian Paul: On 10/29/2012 05:05 PM, srol...@vmware.com wrote: From: Roland Scheideggersrol...@vmware.com Make sure clipping is needed sometimes, and more often use small index counts, to expose issues and excercise more paths in mesa's draw module. ---

Re: [Piglit] [PATCH] tests/spec/arb_robustness/draw-vbo-bounds.c: add clipping

2012-10-30 Thread Roland Scheidegger
Am 30.10.2012 20:01, schrieb Ian Romanick: On 10/29/2012 05:34 PM, Roland Scheidegger wrote: Am 30.10.2012 00:22, schrieb Brian Paul: On 10/29/2012 05:05 PM, srol...@vmware.com wrote: From: Roland Scheideggersrol...@vmware.com Make sure clipping is needed sometimes, and more often use small

Re: [Piglit] [PATCH] fs-texelFetchOffset-2D: don't assume undefined values to be solid black

2012-11-27 Thread Roland Scheidegger
Am 27.11.2012 08:25, schrieb Eric Anholt: srol...@vmware.com writes: From: Roland Scheidegger srol...@vmware.com core GL specifies out-of-bound accesses have undefined behavior, which includes crashes. Crashes are very much undesired, but ARB_robustness still allows undefined values

Re: [Piglit] [PATCH] glsl-1.30: add basic test for testing textureOffset functionality.

2013-03-05 Thread Roland Scheidegger
Am 05.03.2013 19:41, schrieb Brian Paul: On 03/05/2013 11:03 AM, Roland Scheidegger wrote: Am 05.03.2013 17:05, schrieb Brian Paul: On 03/05/2013 08:37 AM, srol...@vmware.com wrote: From: Roland Scheideggersrol...@vmware.com This is pretty rough and doesn't really test all that much (just

Re: [Piglit] [PATCH] glsl-1.30: add basic test for testing textureOffset functionality.

2013-03-22 Thread Roland Scheidegger
Please disregard this sent the wrong stuff... Am 22.03.2013 23:21, schrieb srol...@vmware.com: From: Roland Scheidegger srol...@vmware.com This is pretty rough and doesn't really test all that much (just textureOffsetLod, but there's other texturing functions with offsets), doesn't test

Re: [Piglit] [PATCH] glsl-fs-main-return-conditional: Test for using return in main

2013-03-22 Thread Roland Scheidegger
Am 23.03.2013 01:41, schrieb Ian Romanick: On 03/22/2013 03:28 PM, srol...@vmware.com wrote: From: Roland Scheidegger srol...@vmware.com Similar to glsl-fs-main-return (and glsl-vs-main-return), this is testing using return in main. Contrary to the these other tests, this hits both the cases

Re: [Piglit] [PATCH] arb_shader_texture_lod-texgradcube: Test explicit derivatives for cube maps

2013-04-04 Thread Roland Scheidegger
Am 04.04.2013 01:55, schrieb Brian Paul: On 04/03/2013 05:14 PM, srol...@vmware.com wrote: From: Roland Scheideggersrol...@vmware.com Similar to arb_shader_texture_lod-texgrad, but using cube maps instead. Given the somewhat undefined behavior of explicit gradients with cube maps, the main

Re: [Piglit] [PATCH] arb_shader_texture_lod-texgradcube: Test explicit derivatives for cube maps

2013-04-04 Thread Roland Scheidegger
Am 04.04.2013 17:48, schrieb Brian Paul: On 04/04/2013 09:33 AM, Roland Scheidegger wrote: Am 04.04.2013 01:55, schrieb Brian Paul: On 04/03/2013 05:14 PM, srol...@vmware.com wrote: From: Roland Scheideggersrol...@vmware.com Similar to arb_shader_texture_lod-texgrad, but using cube maps

Re: [Piglit] [PATCH] triangle-rasterization: increase the precision of the test

2013-10-31 Thread Roland Scheidegger
Am 31.10.2013 20:34, schrieb Ian Romanick: On 10/28/2013 04:53 PM, Zack Rusin wrote: Increase the subpixel precision to 8 bits. This requires changing some of the code to 64 bits to avoid overflows. I'm not that familiar with this test. Does this change make this test more strict? Does it

Re: [Piglit] [PATCH 1/3] util: add piglit_array_texture() function

2014-08-05 Thread Roland Scheidegger
piglit_require_transform_feedback(void); For the series: Reviewed-by: Roland Scheidegger srol...@vmware.com ___ Piglit mailing list Piglit@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/piglit

Re: [Piglit] [PATCH] fp-indirections: limit to 1024 indirections

2014-09-18 Thread Roland Scheidegger
Am 18.09.2014 16:16, schrieb Brian Paul: Reviewed-by: Brian Paul bri...@vmware.com Just a minor suggestion below (in 2 places). On 09/18/2014 08:09 AM, srol...@vmware.com wrote: From: Roland Scheidegger srol...@vmware.com The test takes ages (somewhere in the order of an hour

Re: [Piglit] [PATCH 3/4] fp-indirections2: limit max_samples to 256

2014-09-19 Thread Roland Scheidegger
Am 19.09.2014 17:12, schrieb Brian Paul: So that llvmpipe can run in a reasonable amount of time. --- tests/shaders/fp-indirections2.c |5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/tests/shaders/fp-indirections2.c b/tests/shaders/fp-indirections2.c index

Re: [Piglit] [PATCH] glsl-1.50-execution-primitive-id-no-gs-first-vertex: First provoking vertex variation.

2014-12-01 Thread Roland Scheidegger
One nitpick otherwise looks good to me. Reviewed-by: Roland Scheidegger srol...@vmware.com Am 01.12.2014 um 16:57 schrieb jfons...@vmware.com: From: José Fonseca jfons...@vmware.com Adapt the primitive-id-no-gs-first test case to first provoking vertex convention, exposing a bug in llvmpipe

Re: [Piglit] [PATCH] glsl-1.50/execution: add primitive id tests with tri strip primitives

2014-12-09 Thread Roland Scheidegger
ping? Am 06.12.2014 um 05:20 schrieb srol...@vmware.com: From: Roland Scheidegger srol...@vmware.com Just using tri strip instead of tri fan. Exposing more bugs in llvmpipe/draw... --- ...imitive-id-no-gs-strip-first-vertex.shader_test | 62 ++ .../execution

Re: [Piglit] [PATCH] cubemap-mismatch: new test of mismatched cube map faces

2014-12-18 Thread Roland Scheidegger
LGTM, though the test could even be more mean by trying different internal formats too. This is indeed crazy stuff, just an unfortunate side effect of specifying all faces individually. (Makes you wonder why GL didn't use TexImage3D for this just like it does for cube map arrays, but I guess there

Re: [Piglit] [PATCH] cmake: Enforce standard C99 syntax for variadic macros.

2015-02-18 Thread Roland Scheidegger
Am 18.02.2015 um 14:36 schrieb Jose Fonseca: Although the non-standard GCC syntax has some nice properties, for most practical cases the standard C99 syntax is perfectly fine. Particuarly for printf-like macros, which pretty much account for most the uses of variadic macros in piglit.

Re: [Piglit] [PATCH] arb_uniform_buffer_object: Exercise coherent fragment shader UBO.

2015-02-18 Thread Roland Scheidegger
+gl_FragColor = color * color_scale;\n }\n; #define NUM_SQUARES 4 This looks good to me, the test comment says though it's the same as rendering.c except for the persistent mapped ubos which is no longer quite true. Reviewed-by: Roland Scheidegger srol...@vmware.com

Re: [Piglit] [PATCH] arb-provoking-vertex-render: a new test that actually checks all prim types

2015-01-13 Thread Roland Scheidegger
Looks good to me, just a minor nit below. Does not quite check all prim types as it misses the adjacency ones but I guess that would require some more strict gl version check... Reviewed-by: Roland Scheidegger srol...@vmware.com Am 13.01.2015 um 17:57 schrieb Brian Paul: None of the other

Re: [Piglit] [PATCH] all.py: Actually add_texwrap_target_tests's target parameter to the test.

2015-06-18 Thread Roland Scheidegger
', 'bordercolor'], +adder(['texwrap', target, 'GL_RGBA8', 'proj'], + 'texwrap {} proj'.format(target)) +adder(['texwrap', target, 'GL_RGBA8', 'proj', 'bordercolor'], 'texwrap {} proj bordercolor'.format(target)) Reviewed-by: Roland Scheidegger srol...@vmware.com

Re: [Piglit] [PATCH] cmake: Target C99.

2015-12-10 Thread Roland Scheidegger
mode so far without this being an explicit requirement. But I don't have anything against updating that (implicit?) policy to c99 (minus whatever msvc still doesn't support...), albeit it's certainly not me who should say what the requirements need to be. Reviewed-by: Roland Scheidegger <srol...@vmware.com> Roland ___ Piglit mailing list Piglit@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/piglit

Re: [Piglit] [PATCH] polygon-mode-facing: verify facing information is preserved with unfilled prims

2016-01-05 Thread Roland Scheidegger
Am 04.01.2016 um 20:45 schrieb Jose Fonseca: > On 22/12/15 18:05, srol...@vmware.com wrote: >> From: Roland Scheidegger <srol...@vmware.com> >> >> This test is quite mean to mesa's draw pipe. llvmpipe/softpipe fail >> even if >> front and back fill mo

Re: [Piglit] [PATCH] glsl-1.50-geometry-end-primitive: ensure position data is reasonably aligned

2016-01-08 Thread Roland Scheidegger
Oops just forget the changes on the quad-invariance test (this is just what I used to prove the same issue on "real" hw). Am 09.01.2016 um 02:12 schrieb srol...@vmware.com: > From: Roland Scheidegger <srol...@vmware.com> > > The rect halves comparison is in fact not val

Re: [Piglit] [PATCH] glsl-1.50-geometry-end-primitive: ensure position data is reasonably aligned

2016-01-08 Thread Roland Scheidegger
Am 09.01.2016 um 04:09 schrieb Ian Romanick: > On 01/08/2016 05:12 PM, srol...@vmware.com wrote: >> From: Roland Scheidegger <srol...@vmware.com> >> >> The rect halves comparison is in fact not valid in general. The reason is >> that >> the same geometr

Re: [Piglit] [PATCH] polygon-mode-facing: verify facing information is preserved with unfilled prims

2015-12-28 Thread Roland Scheidegger
Ah yes, I'll add that before commiting. Roland Am 23.12.2015 um 02:04 schrieb Dylan Baker: > You'll need to add this to all.py > > On Tue, Dec 22, 2015 at 10:05 AM, <srol...@vmware.com > <mailto:srol...@vmware.com>> wrote: > > From: Roland Scheidegger <srol

Re: [Piglit] [PATCH 4/7] all.py: add arb_framebuffer_srgb-srgb_conformance

2015-12-28 Thread Roland Scheidegger
Am 24.12.2015 um 01:34 schrieb baker.dyla...@gmail.com: > From: Dylan Baker <baker.dyla...@gmail.com> > > This wasn't added to all.py. > > cc: Roland Scheidegger <srol...@vmware.com> > Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> > --- >

Re: [Piglit] [PATCH] arb_texture_view-mipgen: add test to verify correct format is used for mipgen

2016-01-12 Thread Roland Scheidegger
Am 13.01.2016 um 03:32 schrieb Ilia Mirkin: > On Tue, Jan 12, 2016 at 8:07 PM, <srol...@vmware.com> wrote: >> From: Roland Scheidegger <srol...@vmware.com> >> >> At least mesa/st fails this right now (always uses the initial format for mip >> gen). >>

Re: [Piglit] [PATCH] glsl-1.50-geometry-end-primitive: ensure position data is reasonably aligned

2016-01-12 Thread Roland Scheidegger
ping? Am 09.01.2016 um 05:40 schrieb srol...@vmware.com: > From: Roland Scheidegger <srol...@vmware.com> > > The rect halves comparison is in fact not valid in general. The reason is that > the same geometry does not have the same precision even if it it just shifted >

Re: [Piglit] [Mesa-dev] [PATCH] Add (un)packHalf tests which don't fail on GCN

2016-02-05 Thread Roland Scheidegger
Am 05.02.2016 um 16:08 schrieb Marek Olšák: > On Fri, Feb 5, 2016 at 3:56 PM, Roland Scheidegger <srol...@vmware.com> wrote: >> Am 05.02.2016 um 15:44 schrieb Marek Olšák: >>> On Fri, Feb 5, 2016 at 10:57 AM, Marek Olšák <mar...@gmail.com> wrote: >>>> O

Re: [Piglit] [Mesa-dev] [PATCH] Add (un)packHalf tests which don't fail on GCN

2016-02-05 Thread Roland Scheidegger
Am 05.02.2016 um 15:44 schrieb Marek Olšák: > On Fri, Feb 5, 2016 at 10:57 AM, Marek Olšák wrote: >> On Fri, Feb 5, 2016 at 1:55 AM, Matt Turner wrote: >>> On Thu, Feb 4, 2016 at 10:50 AM, Marek Olšák wrote: From: Marek Olšák

Re: [Piglit] [Mesa-dev] [PATCH] Add (un)packHalf tests which don't fail on GCN

2016-02-05 Thread Roland Scheidegger
Am 05.02.2016 um 17:04 schrieb Marek Olšák: > On Fri, Feb 5, 2016 at 4:48 PM, Roland Scheidegger <srol...@vmware.com> wrote: >> Am 05.02.2016 um 16:08 schrieb Marek Olšák: >>> On Fri, Feb 5, 2016 at 3:56 PM, Roland Scheidegger <srol...@vmware.com> >>> wro

Re: [Piglit] [Mesa-dev] [PATCH] Add (un)packHalf tests which don't fail on GCN

2016-02-04 Thread Roland Scheidegger
Am 05.02.2016 um 01:55 schrieb Matt Turner: > On Thu, Feb 4, 2016 at 10:50 AM, Marek Olšák wrote: >> From: Marek Olšák >> >> This is a subset of the generated tests which are known to fail >> on everything except CPU emulation (AFAIK). >> --- > > This is

Re: [Piglit] [PATCH] Add a new GL_ARB_clip_control viewport test (v2)

2016-02-09 Thread Roland Scheidegger
right screen quadrant */ > + glViewport(half_w, 0, half_w, half_h); > + draw_test_pattern(false); > + if (!check_test_pattern(half_w, 0)) > + pass = false; > + > + > + /* Test inverted GL coordinates */ > + glClipControl(GL_UPPER_LEFT, GL_NEGATIVE_ONE_TO_ONE); > + glClear(GL_COLOR_BUFFER_BIT); > + > + /* Inverted GL origin / Draw in upper-left screen quadrant */ > + glViewport(0, half_h, half_w, half_h); > + draw_test_pattern(true); > + if (!check_test_pattern(0, half_h)) > + pass = false; > + > + /* Inverted GL origin / Draw in lower-right screen quadrant */ > + glViewport(half_w, 0, half_w, half_h); > + draw_test_pattern(true); > + if (!check_test_pattern(half_w, 0)) > + pass = false; > + > + piglit_present_results(); > + > + return pass ? PIGLIT_PASS : PIGLIT_FAIL; > +} > For the series: Reviewed-by: Roland Scheidegger <srol...@vmware.com> ___ Piglit mailing list Piglit@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/piglit

Re: [Piglit] [PATCH] gl-1.0-simple-readbuffer: new trivial glReadBuffer() test

2016-04-07 Thread Roland Scheidegger
+void > +piglit_init(int argc, char **argv) > +{ > + glReadBuffer(GL_FRONT); > + > + if (!piglit_check_gl_error(GL_NO_ERROR)) { > + piglit_report_result(PIGLIT_FAIL); > + } > + > + piglit_report_result(PIGLIT_PASS); > +} > Is that a new most simple piglit test ;-). Reviewed-by: Roland Scheidegger <srol...@vmware.com> ___ Piglit mailing list Piglit@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/piglit

Re: [Piglit] [PATCH] tex-miplevel-selection: only require glsl 1.30 for textureOffset 2DArrayShadow

2016-04-19 Thread Roland Scheidegger
Am 19.04.2016 um 18:41 schrieb Jose Fonseca: > On 19/04/16 01:32, srol...@vmware.com wrote: >> From: Roland Scheidegger <srol...@vmware.com> >> >> The spec doesn't really say this should work in older versions. It was >> first >> added in glsl 4.30, ment

Re: [Piglit] [PATCH] tex-miplevel-selection: only require glsl 1.30 for textureOffset 2DArrayShadow

2016-04-19 Thread Roland Scheidegger
Am 19.04.2016 um 19:27 schrieb Jose Fonseca: > On 19/04/16 18:21, Roland Scheidegger wrote: >> Am 19.04.2016 um 18:41 schrieb Jose Fonseca: >>> On 19/04/16 01:32, srol...@vmware.com wrote: >>>> From: Roland Scheidegger <srol...@vmware.com> >>>> &

Re: [Piglit] [PATCH] tests/llvmpipe: Disable gl-1.0-blend-func

2016-09-12 Thread Roland Scheidegger
Am 09.09.2016 um 23:32 schrieb Adam Jackson: > On Fri, 2016-09-09 at 12:09 -0700, Ian Romanick wrote: > >> 16 *gigabytes*? Why? That has to indicate a bug somewhere... >> possibly a leak in the test? > > I suspect more likely a leak in llvmpipe, or in how it's driving llvm. > You're going to

Re: [Piglit] [PATCH] gl-1.3-alpha_to_coverage_nop: new test of no-op GL_SAMPLE_ALPHA_TO_COVERAGE

2016-09-15 Thread Roland Scheidegger
would need to translate the alpha_to_coverage bit away, otherwise it's impossible for a driver to implement both gl and d3d10 behavior. Reviewed-by: Roland Scheidegger <srol...@vmware.com> Am 15.09.2016 um 22:16 schrieb Brian Paul: > Currently fails with llvmpipe. Fragments with alpha <= 0.5 a

Re: [Piglit] [PATCH] gl-3.2/layered-rendering/framebuffertexture: check for MSAA support

2017-08-16 Thread Roland Scheidegger
Thanks for fixing this. Reviewed-by: Roland Scheidegger <srol...@vmware.com> Am 16.08.2017 um 05:30 schrieb Brian Paul: > Skip testing MSAA textures if GL_MAX_SAMPLES = 0. > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102123 > --- > tests/spec/gl-

Re: [Piglit] [PATCH] ext_framebuffer_multisample: add --alpha-to-one option to coverage test

2017-08-04 Thread Roland Scheidegger
schrieb Brian Paul: > It passes here for me w/ NVIDIA's driver but I'll hold off on this until > I can take a closer look. > > -Brina > > On 08/04/2017 11:11 AM, Roland Scheidegger wrote: >> I think should mention that the reference probably isn't quite right (if >>

Re: [Piglit] [PATCH] msaa: add test to verify alpha-to-coverage together with alpha-to-one

2017-08-04 Thread Roland Scheidegger
Am 04.08.2017 um 16:43 schrieb Brian Paul: > On 08/04/2017 08:32 AM, srol...@vmware.com wrote: >> From: Roland Scheidegger <srol...@vmware.com> >> >> I believe the sole reason alpha-to-one exists at all is that it can be >> used together with alpha-to-coverage (ot

Re: [Piglit] [PATCH] ext_framebuffer_multisample: add --alpha-to-one option to coverage test

2017-08-04 Thread Roland Scheidegger
that the samples which get written thanks to coverage-to-alpha also had their alpha values changed to 1.0, but I could be wrong and maybe it is correct after all... In any case, Reviewed-by: Roland Scheidegger <srol...@vmware.com> Am 04.08.2017 um 19:03 schrieb Brian Paul: > Alpha-to

Re: [Piglit] [Review Request (master branch)] srgb_conformance: fix error computation

2017-05-05 Thread Roland Scheidegger
Reviewed-by: Roland Scheidegger <srol...@vmware.com> Am 05.05.2017 um 08:47 schrieb Neha Bhende: > piglit_linear_to_srgb() returns float values in [0,1]. The test was > comparing it against integer values in [0,255]. This is why test was > failing. > > Also, fix the o

Re: [Piglit] [PATCH] arb_texture_query_lod: add tolerance for some comparisons

2017-09-14 Thread Roland Scheidegger
ping? Am 09.09.2017 um 07:31 schrieb srol...@vmware.com: > From: Roland Scheidegger <srol...@vmware.com> > > Tolerance was added for the tests a while ago, but it looks like it was > forgotten for the nearest_biased test (the nearest one has it). > Also, for the linear te

Re: [Piglit] [PATCH] mingw: use default 2MB stack size instead of 1MB

2017-10-12 Thread Roland Scheidegger
Am 12.10.2017 um 21:19 schrieb Brian Paul: > On 10/12/2017 12:11 PM, Jose Fonseca wrote: >> On 12/10/17 17:51, Brian Paul wrote: >>> On 10/12/2017 08:04 AM, Jose Fonseca wrote: The intent here was not so much to match the piglti MSVC build, but apps build by MSVC in general.

Re: [Piglit] [PATCH] fbo-blending-snorm: new test for testing snorm blend behavior

2017-11-22 Thread Roland Scheidegger
Am 22.11.2017 um 19:45 schrieb Eric Anholt: > srol...@vmware.com writes: > >> From: Roland Scheidegger <srol...@vmware.com> >> >> The existing fbo-blending-formats test is mostly useless for anything but >> unorm formats, since it does not test any values ou

Re: [Piglit] [PATCH] fbo-blending-snorm: new test for testing snorm blend behavior

2017-11-22 Thread Roland Scheidegger
Am 22.11.2017 um 20:46 schrieb Ilia Mirkin: > On Wed, Nov 22, 2017 at 2:19 PM, Roland Scheidegger <srol...@vmware.com> > wrote: >> Am 22.11.2017 um 20:17 schrieb Roland Scheidegger: >>> Am 22.11.2017 um 19:45 schrieb Eric Anholt: >>>> srol...@vmware.com writ

Re: [Piglit] [PATCH] fbo-blending-snorm: new test for testing snorm blend behavior

2017-11-22 Thread Roland Scheidegger
Am 22.11.2017 um 20:17 schrieb Roland Scheidegger: > Am 22.11.2017 um 19:45 schrieb Eric Anholt: >> srol...@vmware.com writes: >> >>> From: Roland Scheidegger <srol...@vmware.com> >>> >>> The existing fbo-blending-formats test is mostly useless f

Re: [Piglit] [PATCH] isinf-and-isnan: add clamp / min / max tests

2017-11-13 Thread Roland Scheidegger
Ping? Am 09.11.2017 um 19:11 schrieb srol...@vmware.com: > From: Roland Scheidegger <srol...@vmware.com> > > We expect non-nan results for these (according to d3d10 rules, and at least > for min/max, also according to ieee rules). The tests will not actually fail > with othe

Re: [Piglit] [PATCH 1/3] arb_texture_multisample: don't use hard-coded sample counts

2017-12-05 Thread Roland Scheidegger
LOR_TEXTURE_SAMPLES, _samples); > +if (max_samples < 1) { > + printf("GL_MAX_COLOR_TEXTURE_SAMPLES must be at least one\n"); > + piglit_report_result(PIGLIT_FAIL); > +} Do you want to use the maximum possible or restrict that to something smaller (in case the value retu

Re: [Piglit] [PATCH] draw-vertices-2101010: Accept either SNORM conversion formula.

2017-12-29 Thread Roland Scheidegger
I'm pretty sure that helps llvmpipe too. Reviewed-by: Roland Scheidegger <srol...@vmware.com> Am 26.12.2017 um 07:55 schrieb Kenneth Graunke: > OpenGL defines two equations for converting from signed-normalized > to floating point data. These are: > > f =

Re: [Piglit] [PATCH] arb_texture_multisample: stress test of very large textures (v3)

2017-12-29 Thread Roland Scheidegger
Reviewed-by: Roland Scheidegger <srol...@vmware.com> Am 24.12.2017 um 23:41 schrieb Brian Paul: > Create the largest possible 2D GL_RGBA_32F multisampled texture, load it > with known values the read it back and see if the values match up. > > The --array option runs the test

Re: [Piglit] [PATCH] gl-1.0-blend-func: skip some blend tests when using LLVM 3.8

2018-01-18 Thread Roland Scheidegger
ying to work around it in llvmpipe code, but there was no trivial solution (for not just avoiding the hang but still correctly rendering) and I didn't care enough about that particular llvm version, even more so since it was fixed in the stable branch...) But in any case if you really think it'

Re: [Piglit] [PATCH] gl-1.0-blend-func: skip some blend tests when using LLVM 3.8

2018-01-19 Thread Roland Scheidegger
Am 19.01.2018 um 06:35 schrieb Eric Anholt: > Brian Paul writes: > >> On 01/18/2018 01:27 PM, Eric Anholt wrote: >>> Brian Paul writes: >>> To avoid an infinite loop. See code comments for details. >>> >>> Skipping a failing test and returning pass is

Re: [Piglit] [PATCH] gl-1.0-blend-func: skip some blend tests when using LLVM 3.8

2018-01-18 Thread Roland Scheidegger
Am 18.01.2018 um 21:04 schrieb Brian Paul: > On 01/18/2018 12:38 PM, Roland Scheidegger wrote: >> Am 18.01.2018 um 20:07 schrieb Brian Paul: >>> To avoid an infinite loop.  See code comments for details. >>> --- >>>   tests/spec/gl-1.0/blend.c | 39 +++

Re: [Piglit] [PATCH 3/3] gl-1.0-dlist-materials: new test of materials in display lists

2018-01-18 Thread Roland Scheidegger
+ list = glGenLists(1); > + glNewList(list, GL_COMPILE); > + > + /* > + * Draw tri strip drawing two quads- left=red, right=green. whitespace before "-" ? Ahhh materials. Quite a blast from the past :-). I faintly remember them causing nothing but probl

Re: [Piglit] [PATCH 2/3] gl-1.0-blend-func: add --quick option

2018-01-18 Thread Roland Scheidegger
That should help llvmpipe quite a bit... Albeit 1% doesn't sound like it would give a lot of coverage, maybe a bit more (5% or so) would still cut down the time significantly while having less risk of missing failures? Either way, for 1-2/3 Reviewed-by: Roland Scheidegger <srol...@vmware.com&g

Re: [Piglit] [PATCH] teximage-colors: accept -127 instead of -128 for exact snorm up/download

2018-01-11 Thread Roland Scheidegger
ping? Am 08.01.2018 um 01:20 schrieb srol...@vmware.com: > From: Roland Scheidegger <srol...@vmware.com> > > -128 and -127 represent exactly the same value (-1.0) for snorm8 values, > as do -32768/-32767 for snorm16. Therefore it seems quite reasonable that an > implem

Re: [Piglit] [PATCH 1/2] arb_internalformat_query2: avoid bogus error spam about unsupported pnames

2018-01-26 Thread Roland Scheidegger
Am 27.01.2018 um 02:00 schrieb srol...@vmware.com: > From: Roland Scheidegger <srol...@vmware.com> > > The test fed the equivalent_pname into the get_unsupported_response() > function, which led to nonsense logged errors as this only recognizes > the original pna

Re: [Piglit] [PATCH] arb_sample_shading: Add gl_SampleMaskIn subtest with msaa disabled, fix test

2018-02-05 Thread Roland Scheidegger
Am 05.02.2018 um 17:12 schrieb Brian Paul: > On 02/04/2018 04:03 PM, srol...@vmware.com wrote: >> From: Roland Scheidegger <srol...@vmware.com> >> >> The spec says with msaa disabled gl_SampleMaskIn is always 1. >> This is not particularly related to arb_samp

Re: [Piglit] [PATCH] arb_texture_buffer_object/indexed: test indexed samplers with tbo

2018-01-02 Thread Roland Scheidegger
Am 03.01.2018 um 04:40 schrieb Ilia Mirkin: > Reviewed-by: Ilia Mirkin > > On Tue, Jan 2, 2018 at 9:27 PM, wrote: >> + static const char *fs_source = >> + "#version 150\n" >> + "#extension GL_ARB_gpu_shader5:

Re: [Piglit] [PATCH 1/2] textureSize: add ability to test tess eval stage

2018-01-03 Thread Roland Scheidegger
est should be added to all.py (in > multiple places). > > Other than that, series is > Reviewed-by: Fabian Bieler <fabianbie...@fastmail.fm> > > On 2018-01-01 06:41, srol...@vmware.com wrote: >> From: Roland Scheidegger <srol...@vmware.com> >> >> Th

Re: [Piglit] [PATCH] Add tests for GL_EXTENSION_STRING vs. old idTech2 / idTech3 games

2018-09-24 Thread Roland Scheidegger
Am 24.09.2018 um 15:56 schrieb Ian Romanick: > From: Ian Romanick > > As people dig up other games, we can (and should) easily add them. > > Signed-off-by: Ian Romanick > Cc: Federico Dossena > Cc: Roland Scheidegger > --- > tests/general/CMakeLists.gl.txt

Re: [Piglit] [PATCH] appveyor: Update instructions for personal repositories.

2019-03-14 Thread Roland Scheidegger
Looks good to me. For the series: Reviewed-by: Roland Scheidegger Am 13.03.19 um 15:16 schrieb Jose Fonseca: > With GitLab is now much easier to setup. > --- > appveyor.yml | 22 -- > 1 file changed, 8 insertions(+), 14 deletions(-) > > diff --gi