<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>