Mesa (master): i965: Remove GL_ARB_color_buffer_float from GL core contexts .

Eric Anholt anholt at kemper.freedesktop.org
Mon May 6 16:03:35 UTC 2013


Module: Mesa
Branch: master
Commit: bd850cb4f2c77e2eb6716c865c40b9976633fc23
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=bd850cb4f2c77e2eb6716c865c40b9976633fc23

Author: Eric Anholt <eric at anholt.net>
Date:   Thu Apr 25 12:06:23 2013 -0700

i965: Remove GL_ARB_color_buffer_float from GL core contexts.

Of the 3 controls in the extension, one was kept in GL core and the other
two were explicitly deprecated and the reasonable default behavior was
encoded in the spec.  By not exposing the extension, we avoid shader
recompiles when switching between float and unorm color buffers.

Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>

---

 src/mesa/drivers/dri/intel/intel_extensions.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/src/mesa/drivers/dri/intel/intel_extensions.c b/src/mesa/drivers/dri/intel/intel_extensions.c
index 18f19b8..8d8e325 100644
--- a/src/mesa/drivers/dri/intel/intel_extensions.c
+++ b/src/mesa/drivers/dri/intel/intel_extensions.c
@@ -125,7 +125,8 @@ intelInitExtensions(struct gl_context *ctx)
    if (intel->gen >= 4) {
       if (ctx->API == API_OPENGL_CORE)
          ctx->Extensions.ARB_base_instance = true;
-      ctx->Extensions.ARB_color_buffer_float = true;
+      if (ctx->API != API_OPENGL_CORE)
+         ctx->Extensions.ARB_color_buffer_float = true;
       ctx->Extensions.ARB_depth_buffer_float = true;
       ctx->Extensions.ARB_depth_clamp = true;
       ctx->Extensions.ARB_draw_instanced = true;




More information about the mesa-commit mailing list