<div dir="ltr">I just finished crawling through this code and I'm pretty sure we do mostly the right thing for ES.  I'll be even more sure after my series lands.<br><div><br>Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 12, 2016 at 6:31 PM, Kenneth Graunke <span dir="ltr"><<a href="mailto:kenneth@whitecape.org" target="_blank">kenneth@whitecape.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This enables:<br>
- GL_OES_sample_shading<br>
- GL_OES_sample_variables<br>
- GL_OES_shader_multisample_interpolation<br>
<br>
We pass all the CTS tests, and all but 8 of the dEQP-GLES31 tests.<br>
Half of the failing dEQP tests appear to be broken tests; the other<br>
four still need investigating but cover an obscure corner case.<br>
<br>
Otherwise, the functionality is in great shape, and actually in better<br>
shape than the ARB_sample_shading support we've shipped for years<br>
(we fixed a number of bugs when enabling this).  So it makes sense to<br>
enable it for our users in the Mesa 12.0 release.<br>
<br>
Signed-off-by: Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>><br>
---<br>
 src/mesa/drivers/dri/i965/intel_extensions.c | 1 +<br>
 1 file changed, 1 insertion(+)<br>
<br>
diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c b/src/mesa/drivers/dri/i965/intel_extensions.c<br>
index 8d98788..7f44c1d 100644<br>
--- a/src/mesa/drivers/dri/i965/intel_extensions.c<br>
+++ b/src/mesa/drivers/dri/i965/intel_extensions.c<br>
@@ -308,6 +308,7 @@ intelInitExtensions(struct gl_context *ctx)<br>
       ctx->Extensions.EXT_framebuffer_multisample_blit_scaled = true;<br>
       ctx->Extensions.EXT_transform_feedback = true;<br>
       ctx->Extensions.OES_depth_texture_cube_map = true;<br>
+      ctx->Extensions.OES_sample_variables = true;<br>
<br>
       ctx->Extensions.ARB_timer_query = brw->intelScreen->hw_has_timestamp;<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
2.8.2<br>
<br>
_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</font></span></blockquote></div><br></div>