<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 13, 2015 at 11:49 PM, Samuel Iglesias Gonsalvez <span dir="ltr"><<a href="mailto:siglesias@igalia.com" target="_blank">siglesias@igalia.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Packed types are not explicitly allowed to work with GL_ABGR_EXT format in<br>
the OpenGL spec nor GL_EXT_abgr spec.<br>
<br>
NVIDIA allows it but AMD doesn't and Intel driver hasn't allowed it with<br>
UNSIGNED_SHORT_5_5_5_1 and UNSIGNED_INT_10_10_10_2 as of c471b09bf4.<br>
<br>
Signed-off-by: Samuel Iglesias Gonsalvez <<a href="mailto:siglesias@igalia.com">siglesias@igalia.com</a>><br>
---<br>
 tests/glean/tpixelformats.cpp | 6 ++++++<br>
 1 file changed, 6 insertions(+)<br>
<br>
diff --git a/tests/glean/tpixelformats.cpp b/tests/glean/tpixelformats.cpp<br>
index 27881bf..b040957 100644<br>
--- a/tests/glean/tpixelformats.cpp<br>
+++ b/tests/glean/tpixelformats.cpp<br>
@@ -612,6 +612,12 @@ PixelFormatsTest::CompatibleFormatAndType(GLenum format, GLenum datatype) const<br>
        if (format == GL_ABGR_EXT && !haveABGR)<br>
                return false;<br>
<br>
+       // Special case: GL_ABGR_EXT can't be used with packed types<br>
+       // because they are not explicitely allowed by GL_ABGR_EXT spec or<br>
+       // OpenGL spec.<br></blockquote><div><br></div><div>I'd word this differently:<br><br></div><div>...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.<br><br></div><div>Either way,<br></div><div>Reviewed-by: Jason Ekstrand <<a href="mailto:jason.ekstrand@intel.com">jason.ekstrand@intel.com</a>><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+       if (format == GL_ABGR_EXT && IsPackedType(datatype))<br>
+               return false;<br>
+<br>
        if (format == GL_RG && !haveRG)<br>
                return false;<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
2.1.0<br>
<br>
_______________________________________________<br>
Piglit mailing list<br>
<a href="mailto:Piglit@lists.freedesktop.org">Piglit@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/piglit" target="_blank">http://lists.freedesktop.org/mailman/listinfo/piglit</a><br>
</font></span></blockquote></div><br></div></div>