<html>
<head>
<base href="https://bugs.freedesktop.org/">
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - Incorrect simultaneous work of GL_ARB_indirect_parameters and GL_NV_conditional_render."
href="https://bugs.freedesktop.org/show_bug.cgi?id=108759#c3">Comment # 3</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - Incorrect simultaneous work of GL_ARB_indirect_parameters and GL_NV_conditional_render."
href="https://bugs.freedesktop.org/show_bug.cgi?id=108759">bug 108759</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 Illia Iorin from <a href="show_bug.cgi?id=108759#c1">comment #1</a>)
<span class="quote">> Piglit test can found here: <a href="https://patchwork.freedesktop.org/patch/262127/">https://patchwork.freedesktop.org/patch/262127/</a></span >
I lost most of my mail box, so I don't have this patch to reply to. I have two
pieces of feedback.
1. The test requires OpenGL 3.2, so it should use glBeginConditionalRender
instead of glBeginConditionalRenderNV. Drop the requirement for the NV
extension. Conditional rendering was made part of the GL spec in 3.0. It
should also mention "conditional rendering" in the description instead of
GL_NV_conditional_render. I think I gave similar feedback earlier today on the
Mesa MR.
2. The test uses atomic counters, but it does not check that extension. I
doubt any driver will ever support GL_ARB_indirect_parameters and not
GL_ARB_shader_atomic_counters, but still. :)</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the QA Contact for the bug.</li>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>