Re: [Piglit] [PATCH] glean/pixelformats: restrict GL_ABGR_EXT format test to the types in the spec
On Wednesday, January 14, 2015 10:06:00 AM Jason Ekstrand wrote: > On Tue, Jan 13, 2015 at 11:49 PM, Samuel Iglesias Gonsalvez < > > sigles...@igalia.com> wrote: > > Packed types are not explicitly allowed to work with GL_ABGR_EXT format in > > the OpenGL spec nor GL_EXT_abgr spec. > > > > NVIDIA allows it but AMD doesn't and Intel driver hasn't allowed it with > > UNSIGNED_SHORT_5_5_5_1 and UNSIGNED_INT_10_10_10_2 as of c471b09bf4. > > > > Signed-off-by: Samuel Iglesias Gonsalvez > > --- > > > > tests/glean/tpixelformats.cpp | 6 ++ > > 1 file changed, 6 insertions(+) > > > > diff --git a/tests/glean/tpixelformats.cpp b/tests/glean/tpixelformats.cpp > > index 27881bf..b040957 100644 > > --- a/tests/glean/tpixelformats.cpp > > +++ b/tests/glean/tpixelformats.cpp > > @@ -612,6 +612,12 @@ PixelFormatsTest::CompatibleFormatAndType(GLenum > > format, GLenum datatype) const > > > > if (format == GL_ABGR_EXT && !haveABGR) > > > > return false; > > > > + // Special case: GL_ABGR_EXT can't be used with packed types > > + // because they are not explicitely allowed by GL_ABGR_EXT spec or > > + // OpenGL spec. > > I'd word this differently: > > ...because the packed formats specs (which were all written after the > GL_EXT_abgr) explicitly say that the packed formats can only be used with > GL_RGB, GL_BGR, GL_RGBA, or GL_BGRA and do not mention GL_ABGR_EXT. > > Either way, > Reviewed-by: Jason Ekstrand > Thanks Jason! Sam > > + if (format == GL_ABGR_EXT && IsPackedType(datatype)) > > + return false; > > + > > > > if (format == GL_RG && !haveRG) > > > > return false; > > > > -- > > 2.1.0 > > > > ___ > > Piglit mailing list > > Piglit@lists.freedesktop.org > > http://lists.freedesktop.org/mailman/listinfo/piglit signature.asc Description: This is a digitally signed message part. ___ Piglit mailing list Piglit@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/piglit
Re: [Piglit] [PATCH] gpu_shader5: Fix the GLSL version in indirect-projGradOffset-shadow
Reviewed-by: Matt Turner ___ Piglit mailing list Piglit@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/piglit
[Piglit] [PATCH] gpu_shader5: Fix the GLSL version in indirect-projGradOffset-shadow
The ARB_gpu_shader5 extension requires GLSL 1.50 and we were only requiring GLSL 1.30 --- .../spec/arb_gpu_shader5/compiler/indirect-projGradOffset-shadow.frag | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/spec/arb_gpu_shader5/compiler/indirect-projGradOffset-shadow.frag b/tests/spec/arb_gpu_shader5/compiler/indirect-projGradOffset-shadow.frag index ccf1886..aaeeba4 100644 --- a/tests/spec/arb_gpu_shader5/compiler/indirect-projGradOffset-shadow.frag +++ b/tests/spec/arb_gpu_shader5/compiler/indirect-projGradOffset-shadow.frag @@ -1,9 +1,9 @@ // [config] // expect_result: pass -// glsl_version: 1.30 +// glsl_version: 1.50 // require_extensions: GL_ARB_gpu_shader5 // [end config] -#version 130 +#version 150 #extension GL_ARB_gpu_shader5: require uniform int i; -- 2.2.1 ___ Piglit mailing list Piglit@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/piglit
Re: [Piglit] [PATCH] glean/pixelformats: restrict GL_ABGR_EXT format test to the types in the spec
On Tue, Jan 13, 2015 at 11:49 PM, Samuel Iglesias Gonsalvez < sigles...@igalia.com> wrote: > Packed types are not explicitly allowed to work with GL_ABGR_EXT format in > the OpenGL spec nor GL_EXT_abgr spec. > > NVIDIA allows it but AMD doesn't and Intel driver hasn't allowed it with > UNSIGNED_SHORT_5_5_5_1 and UNSIGNED_INT_10_10_10_2 as of c471b09bf4. > > Signed-off-by: Samuel Iglesias Gonsalvez > --- > tests/glean/tpixelformats.cpp | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/tests/glean/tpixelformats.cpp b/tests/glean/tpixelformats.cpp > index 27881bf..b040957 100644 > --- a/tests/glean/tpixelformats.cpp > +++ b/tests/glean/tpixelformats.cpp > @@ -612,6 +612,12 @@ PixelFormatsTest::CompatibleFormatAndType(GLenum > format, GLenum datatype) const > if (format == GL_ABGR_EXT && !haveABGR) > return false; > > + // Special case: GL_ABGR_EXT can't be used with packed types > + // because they are not explicitely allowed by GL_ABGR_EXT spec or > + // OpenGL spec. > I'd word this differently: ...because the packed formats specs (which were all written after the GL_EXT_abgr) explicitly say that the packed formats can only be used with GL_RGB, GL_BGR, GL_RGBA, or GL_BGRA and do not mention GL_ABGR_EXT. Either way, Reviewed-by: Jason Ekstrand > + if (format == GL_ABGR_EXT && IsPackedType(datatype)) > + return false; > + > if (format == GL_RG && !haveRG) > return false; > > -- > 2.1.0 > > ___ > Piglit mailing list > Piglit@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/piglit > ___ Piglit mailing list Piglit@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/piglit