<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW --- - dispatch_sanity test lumps GL4.0+ functions into 4.3"
href="https://bugs.freedesktop.org/show_bug.cgi?id=62612#c1">Comment # 1</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW --- - dispatch_sanity test lumps GL4.0+ functions into 4.3"
href="https://bugs.freedesktop.org/show_bug.cgi?id=62612">bug 62612</a>
from <span class="vcard"><a class="email" href="mailto:idr@freedesktop.org" title="Ian Romanick <idr@freedesktop.org>"> <span class="fn">Ian Romanick</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=62612#c0">comment #0</a>)
<span class="quote">> I noticed:
>
> { "glDrawTransformFeedback", 43, -1 },
> { "glDrawTransformFeedbackStream", 43, -1 },
>
> which are from ARB_transform_feedback2 and 3 respectively are in GL 4.0 but
> the test says they must exist by 4.3.</span >
I believe that Jordan did that because Mesa already supports those extensions
via the extensions. Any driver that supports OpenGL 4.0 will populate those
dispatch pointers.
<span class="quote">> I started to fix this, but then noticed that we don't have lists for 4.0,
> 4.1, or 4.2, so the problem is probably much larger than just these two
> extensions.
>
> Maybe we should strip out all of the lines that say "Add to xml" and fix up
> the others.
>
> I also notice
>
> /* GL_ARB_internalformat_query */
> { "glGetInternalformativ", 30, -1 },
>
> but ARB_internalformat_query is part of 4.2 and not a required part of 3.0.
> I think there's some general confusion about what the version field is
> supposed to mean.</span >
However, every driver supports the extension, so the function is in the
dispatch table.
<span class="quote">> IIRC, this is why Ian suggested having separate structs for each version,
> like how GL ES 2 and 3 are handled.</span >
I'm not a huge fan of the version field in the dispatch sanity tests. :) But I
don't think that will solve this particular problem.
I think when we add the next big batch of functions (probably for
ARB_geometry_shader4), we should refactor this test a bit.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>