i965: Remove unused code and delete file.
authorRafael Antognolli <rafael.antognolli@intel.com>
Tue, 30 May 2017 22:08:17 +0000 (15:08 -0700)
committerRafael Antognolli <rafael.antognolli@intel.com>
Thu, 22 Jun 2017 23:51:51 +0000 (16:51 -0700)
The sampler state code was all moved to genxml, so we can get rid of these
functions and delete the file.

Signed-off-by: Rafael Antognolli <rafael.antognolli@intel.com>
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
src/mesa/drivers/dri/i965/Makefile.sources
src/mesa/drivers/dri/i965/brw_sampler_state.c [deleted file]

index 8bac8039ce26d247b89d94b559d9ee29fff0194a..405d67f87d7df93dfad92c21857d34af890dd4d1 100644 (file)
@@ -43,7 +43,6 @@ i965_FILES = \
        brw_primitive_restart.c \
        brw_queryobj.c \
        brw_reset.c \
-       brw_sampler_state.c \
        brw_sf.c \
        brw_state_batch.c \
        brw_state.h \
diff --git a/src/mesa/drivers/dri/i965/brw_sampler_state.c b/src/mesa/drivers/dri/i965/brw_sampler_state.c
deleted file mode 100644 (file)
index 4ac8e94..0000000
+++ /dev/null
@@ -1,625 +0,0 @@
-/*
- Copyright (C) Intel Corp.  2006.  All Rights Reserved.
- Intel funded Tungsten Graphics to
- develop this 3D driver.
-
- Permission is hereby granted, free of charge, to any person obtaining
- a copy of this software and associated documentation files (the
- "Software"), to deal in the Software without restriction, including
- without limitation the rights to use, copy, modify, merge, publish,
- distribute, sublicense, and/or sell copies of the Software, and to
- permit persons to whom the Software is furnished to do so, subject to
- the following conditions:
-
- The above copyright notice and this permission notice (including the
- next paragraph) shall be included in all copies or substantial
- portions of the Software.
-
- THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
- EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
- MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
- IN NO EVENT SHALL THE COPYRIGHT OWNER(S) AND/OR ITS SUPPLIERS BE
- LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
- OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
- WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
-
- **********************************************************************/
- /*
-  * Authors:
-  *   Keith Whitwell <keithw@vmware.com>
-  */
-
-/**
- * @file brw_sampler_state.c
- *
- * This file contains code for emitting SAMPLER_STATE structures, which
- * specifies filter modes, wrap modes, border color, and so on.
- */
-
-#include "brw_context.h"
-#include "brw_state.h"
-#include "brw_defines.h"
-#include "intel_batchbuffer.h"
-#include "intel_mipmap_tree.h"
-
-#include "main/macros.h"
-#include "main/samplerobj.h"
-#include "util/half_float.h"
-
-/**
- * Emit a 3DSTATE_SAMPLER_STATE_POINTERS_{VS,HS,GS,DS,PS} packet.
- */
-static void
-gen7_emit_sampler_state_pointers_xs(struct brw_context *brw,
-                                    struct brw_stage_state *stage_state)
-{
-   static const uint16_t packet_headers[] = {
-      [MESA_SHADER_VERTEX] = _3DSTATE_SAMPLER_STATE_POINTERS_VS,
-      [MESA_SHADER_TESS_CTRL] = _3DSTATE_SAMPLER_STATE_POINTERS_HS,
-      [MESA_SHADER_TESS_EVAL] = _3DSTATE_SAMPLER_STATE_POINTERS_DS,
-      [MESA_SHADER_GEOMETRY] = _3DSTATE_SAMPLER_STATE_POINTERS_GS,
-      [MESA_SHADER_FRAGMENT] = _3DSTATE_SAMPLER_STATE_POINTERS_PS,
-   };
-
-   /* Ivybridge requires a workaround flush before VS packets. */
-   if (brw->gen == 7 && !brw->is_haswell && !brw->is_baytrail &&
-       stage_state->stage == MESA_SHADER_VERTEX) {
-      gen7_emit_vs_workaround_flush(brw);
-   }
-
-   BEGIN_BATCH(2);
-   OUT_BATCH(packet_headers[stage_state->stage] << 16 | (2 - 2));
-   OUT_BATCH(stage_state->sampler_offset);
-   ADVANCE_BATCH();
-}
-
-/**
- * Emit a SAMPLER_STATE structure, given all the fields.
- */
-void
-brw_emit_sampler_state(struct brw_context *brw,
-                       uint32_t *ss,
-                       uint32_t batch_offset_for_sampler_state,
-                       unsigned min_filter,
-                       unsigned mag_filter,
-                       unsigned mip_filter,
-                       unsigned max_anisotropy,
-                       unsigned address_rounding,
-                       unsigned wrap_s,
-                       unsigned wrap_t,
-                       unsigned wrap_r,
-                       unsigned base_level,
-                       unsigned min_lod,
-                       unsigned max_lod,
-                       int lod_bias,
-                       unsigned shadow_function,
-                       bool non_normalized_coordinates,
-                       uint32_t border_color_offset)
-{
-   ss[0] = BRW_SAMPLER_LOD_PRECLAMP_ENABLE |
-           SET_FIELD(mip_filter, BRW_SAMPLER_MIP_FILTER) |
-           SET_FIELD(mag_filter, BRW_SAMPLER_MAG_FILTER) |
-           SET_FIELD(min_filter, BRW_SAMPLER_MIN_FILTER);
-
-   ss[2] = border_color_offset;
-   if (brw->gen < 6) {
-      ss[2] += brw->batch.bo->offset64; /* reloc */
-      brw_emit_reloc(&brw->batch, batch_offset_for_sampler_state + 8,
-                     brw->batch.bo, border_color_offset,
-                     I915_GEM_DOMAIN_SAMPLER, 0);
-   }
-
-   ss[3] = SET_FIELD(max_anisotropy, BRW_SAMPLER_MAX_ANISOTROPY) |
-           SET_FIELD(address_rounding, BRW_SAMPLER_ADDRESS_ROUNDING);
-
-   if (brw->gen >= 7) {
-      ss[0] |= SET_FIELD(lod_bias & 0x1fff, GEN7_SAMPLER_LOD_BIAS);
-
-      if (min_filter == BRW_MAPFILTER_ANISOTROPIC)
-         ss[0] |= GEN7_SAMPLER_EWA_ANISOTROPIC_ALGORITHM;
-
-      ss[1] = SET_FIELD(min_lod, GEN7_SAMPLER_MIN_LOD) |
-              SET_FIELD(max_lod, GEN7_SAMPLER_MAX_LOD) |
-              SET_FIELD(shadow_function, GEN7_SAMPLER_SHADOW_FUNCTION);
-
-      ss[3] |= SET_FIELD(wrap_s, BRW_SAMPLER_TCX_WRAP_MODE) |
-               SET_FIELD(wrap_t, BRW_SAMPLER_TCY_WRAP_MODE) |
-               SET_FIELD(wrap_r, BRW_SAMPLER_TCZ_WRAP_MODE);
-
-      if (non_normalized_coordinates)
-         ss[3] |= GEN7_SAMPLER_NON_NORMALIZED_COORDINATES;
-   } else {
-      ss[0] |= SET_FIELD(lod_bias & 0x7ff, GEN4_SAMPLER_LOD_BIAS) |
-               SET_FIELD(shadow_function, GEN4_SAMPLER_SHADOW_FUNCTION);
-
-      /* This field has existed since the original i965, but is declared MBZ
-       * until Sandy Bridge.  According to the PRM:
-       *
-       *    "This was added to match OpenGL semantics"
-       *
-       * In particular, OpenGL allowed you to offset by 0.5 in certain cases
-       * to get slightly better filtering.  On Ivy Bridge and above, it
-       * appears that this is added to RENDER_SURFACE_STATE::SurfaceMinLOD so
-       * the right value is 0.0 or 0.5 (if you want the wacky behavior).  On
-       * Sandy Bridge, however, this sum does not seem to occur and you have
-       * to set it to the actual base level of the texture.
-       */
-      if (brw->gen == 6)
-         ss[0] |= SET_FIELD(base_level, BRW_SAMPLER_BASE_MIPLEVEL);
-
-      if (brw->gen == 6 && min_filter != mag_filter)
-         ss[0] |= GEN6_SAMPLER_MIN_MAG_NOT_EQUAL;
-
-      ss[1] = SET_FIELD(min_lod, GEN4_SAMPLER_MIN_LOD) |
-              SET_FIELD(max_lod, GEN4_SAMPLER_MAX_LOD) |
-              SET_FIELD(wrap_s, BRW_SAMPLER_TCX_WRAP_MODE) |
-              SET_FIELD(wrap_t, BRW_SAMPLER_TCY_WRAP_MODE) |
-              SET_FIELD(wrap_r, BRW_SAMPLER_TCZ_WRAP_MODE);
-
-      if (brw->gen >= 6 && non_normalized_coordinates)
-         ss[3] |= GEN6_SAMPLER_NON_NORMALIZED_COORDINATES;
-   }
-}
-
-static uint32_t
-translate_wrap_mode(struct brw_context *brw, GLenum wrap, bool using_nearest)
-{
-   switch (wrap) {
-   case GL_REPEAT:
-      return BRW_TEXCOORDMODE_WRAP;
-   case GL_CLAMP:
-      /* GL_CLAMP is the weird mode where coordinates are clamped to
-       * [0.0, 1.0], so linear filtering of coordinates outside of
-       * [0.0, 1.0] give you half edge texel value and half border
-       * color.
-       *
-       * Gen8+ supports this natively.
-       */
-      if (brw->gen >= 8)
-         return GEN8_TEXCOORDMODE_HALF_BORDER;
-
-      /* On Gen4-7.5, we clamp the coordinates in the fragment shader
-       * and set clamp_border here, which gets the result desired.
-       * We just use clamp(_to_edge) for nearest, because for nearest
-       * clamping to 1.0 gives border color instead of the desired
-       * edge texels.
-       */
-      if (using_nearest)
-        return BRW_TEXCOORDMODE_CLAMP;
-      else
-        return BRW_TEXCOORDMODE_CLAMP_BORDER;
-   case GL_CLAMP_TO_EDGE:
-      return BRW_TEXCOORDMODE_CLAMP;
-   case GL_CLAMP_TO_BORDER:
-      return BRW_TEXCOORDMODE_CLAMP_BORDER;
-   case GL_MIRRORED_REPEAT:
-      return BRW_TEXCOORDMODE_MIRROR;
-   case GL_MIRROR_CLAMP_TO_EDGE:
-      return BRW_TEXCOORDMODE_MIRROR_ONCE;
-   default:
-      return BRW_TEXCOORDMODE_WRAP;
-   }
-}
-
-/**
- * Return true if the given wrap mode requires the border color to exist.
- */
-static bool
-wrap_mode_needs_border_color(unsigned wrap_mode)
-{
-   return wrap_mode == BRW_TEXCOORDMODE_CLAMP_BORDER ||
-          wrap_mode == GEN8_TEXCOORDMODE_HALF_BORDER;
-}
-
-static bool
-has_component(mesa_format format, int i)
-{
-   if (_mesa_is_format_color_format(format))
-      return _mesa_format_has_color_component(format, i);
-
-   /* depth and stencil have only one component */
-   return i == 0;
-}
-
-/**
- * Upload SAMPLER_BORDER_COLOR_STATE.
- */
-static void
-upload_default_color(struct brw_context *brw,
-                     const struct gl_sampler_object *sampler,
-                     mesa_format format, GLenum base_format,
-                     bool is_integer_format, bool is_stencil_sampling,
-                     uint32_t *sdc_offset)
-{
-   union gl_color_union color;
-
-   switch (base_format) {
-   case GL_DEPTH_COMPONENT:
-      /* GL specs that border color for depth textures is taken from the
-       * R channel, while the hardware uses A.  Spam R into all the
-       * channels for safety.
-       */
-      color.ui[0] = sampler->BorderColor.ui[0];
-      color.ui[1] = sampler->BorderColor.ui[0];
-      color.ui[2] = sampler->BorderColor.ui[0];
-      color.ui[3] = sampler->BorderColor.ui[0];
-      break;
-   case GL_ALPHA:
-      color.ui[0] = 0u;
-      color.ui[1] = 0u;
-      color.ui[2] = 0u;
-      color.ui[3] = sampler->BorderColor.ui[3];
-      break;
-   case GL_INTENSITY:
-      color.ui[0] = sampler->BorderColor.ui[0];
-      color.ui[1] = sampler->BorderColor.ui[0];
-      color.ui[2] = sampler->BorderColor.ui[0];
-      color.ui[3] = sampler->BorderColor.ui[0];
-      break;
-   case GL_LUMINANCE:
-      color.ui[0] = sampler->BorderColor.ui[0];
-      color.ui[1] = sampler->BorderColor.ui[0];
-      color.ui[2] = sampler->BorderColor.ui[0];
-      color.ui[3] = float_as_int(1.0);
-      break;
-   case GL_LUMINANCE_ALPHA:
-      color.ui[0] = sampler->BorderColor.ui[0];
-      color.ui[1] = sampler->BorderColor.ui[0];
-      color.ui[2] = sampler->BorderColor.ui[0];
-      color.ui[3] = sampler->BorderColor.ui[3];
-      break;
-   default:
-      color.ui[0] = sampler->BorderColor.ui[0];
-      color.ui[1] = sampler->BorderColor.ui[1];
-      color.ui[2] = sampler->BorderColor.ui[2];
-      color.ui[3] = sampler->BorderColor.ui[3];
-      break;
-   }
-
-   /* In some cases we use an RGBA surface format for GL RGB textures,
-    * where we've initialized the A channel to 1.0.  We also have to set
-    * the border color alpha to 1.0 in that case.
-    */
-   if (base_format == GL_RGB)
-      color.ui[3] = float_as_int(1.0);
-
-   if (brw->gen >= 8) {
-      /* On Broadwell, the border color is represented as four 32-bit floats,
-       * integers, or unsigned values, interpreted according to the surface
-       * format.  This matches the sampler->BorderColor union exactly; just
-       * memcpy the values.
-       */
-      uint32_t *sdc = brw_state_batch(brw, 4 * 4, 64, sdc_offset);
-      memcpy(sdc, color.ui, 4 * 4);
-   } else if (brw->is_haswell && (is_integer_format || is_stencil_sampling)) {
-      /* Haswell's integer border color support is completely insane:
-       * SAMPLER_BORDER_COLOR_STATE is 20 DWords.  The first four are
-       * for float colors.  The next 12 DWords are MBZ and only exist to
-       * pad it out to a 64 byte cacheline boundary.  DWords 16-19 then
-       * contain integer colors; these are only used if SURFACE_STATE
-       * has the "Integer Surface Format" bit set.  Even then, the
-       * arrangement of the RGBA data devolves into madness.
-       */
-      uint32_t *sdc = brw_state_batch(brw, 20 * 4, 512, sdc_offset);
-      memset(sdc, 0, 20 * 4);
-      sdc = &sdc[16];
-
-      bool stencil = format == MESA_FORMAT_S_UINT8 || is_stencil_sampling;
-      const int bits_per_channel =
-         _mesa_get_format_bits(format, stencil ? GL_STENCIL_BITS : GL_RED_BITS);
-
-      /* From the Haswell PRM, "Command Reference: Structures", Page 36:
-       * "If any color channel is missing from the surface format,
-       *  corresponding border color should be programmed as zero and if
-       *  alpha channel is missing, corresponding Alpha border color should
-       *  be programmed as 1."
-       */
-      unsigned c[4] = { 0, 0, 0, 1 };
-      for (int i = 0; i < 4; i++) {
-         if (has_component(format, i))
-            c[i] = color.ui[i];
-      }
-
-      switch (bits_per_channel) {
-      case 8:
-         /* Copy RGBA in order. */
-         for (int i = 0; i < 4; i++)
-            ((uint8_t *) sdc)[i] = c[i];
-         break;
-      case 10:
-         /* R10G10B10A2_UINT is treated like a 16-bit format. */
-      case 16:
-         ((uint16_t *) sdc)[0] = c[0]; /* R -> DWord 0, bits 15:0  */
-         ((uint16_t *) sdc)[1] = c[1]; /* G -> DWord 0, bits 31:16 */
-         /* DWord 1 is Reserved/MBZ! */
-         ((uint16_t *) sdc)[4] = c[2]; /* B -> DWord 2, bits 15:0  */
-         ((uint16_t *) sdc)[5] = c[3]; /* A -> DWord 3, bits 31:16 */
-         break;
-      case 32:
-         if (base_format == GL_RG) {
-            /* Careful inspection of the tables reveals that for RG32 formats,
-             * the green channel needs to go where blue normally belongs.
-             */
-            sdc[0] = c[0];
-            sdc[2] = c[1];
-            sdc[3] = 1;
-         } else {
-            /* Copy RGBA in order. */
-            for (int i = 0; i < 4; i++)
-               sdc[i] = c[i];
-         }
-         break;
-      default:
-         assert(!"Invalid number of bits per channel in integer format.");
-         break;
-      }
-   } else if (brw->gen == 5 || brw->gen == 6) {
-      struct gen5_sampler_default_color *sdc;
-
-      sdc = brw_state_batch(brw, sizeof(*sdc), 32, sdc_offset);
-
-      memset(sdc, 0, sizeof(*sdc));
-
-      UNCLAMPED_FLOAT_TO_UBYTE(sdc->ub[0], color.f[0]);
-      UNCLAMPED_FLOAT_TO_UBYTE(sdc->ub[1], color.f[1]);
-      UNCLAMPED_FLOAT_TO_UBYTE(sdc->ub[2], color.f[2]);
-      UNCLAMPED_FLOAT_TO_UBYTE(sdc->ub[3], color.f[3]);
-
-      UNCLAMPED_FLOAT_TO_USHORT(sdc->us[0], color.f[0]);
-      UNCLAMPED_FLOAT_TO_USHORT(sdc->us[1], color.f[1]);
-      UNCLAMPED_FLOAT_TO_USHORT(sdc->us[2], color.f[2]);
-      UNCLAMPED_FLOAT_TO_USHORT(sdc->us[3], color.f[3]);
-
-      UNCLAMPED_FLOAT_TO_SHORT(sdc->s[0], color.f[0]);
-      UNCLAMPED_FLOAT_TO_SHORT(sdc->s[1], color.f[1]);
-      UNCLAMPED_FLOAT_TO_SHORT(sdc->s[2], color.f[2]);
-      UNCLAMPED_FLOAT_TO_SHORT(sdc->s[3], color.f[3]);
-
-      sdc->hf[0] = _mesa_float_to_half(color.f[0]);
-      sdc->hf[1] = _mesa_float_to_half(color.f[1]);
-      sdc->hf[2] = _mesa_float_to_half(color.f[2]);
-      sdc->hf[3] = _mesa_float_to_half(color.f[3]);
-
-      sdc->b[0] = sdc->s[0] >> 8;
-      sdc->b[1] = sdc->s[1] >> 8;
-      sdc->b[2] = sdc->s[2] >> 8;
-      sdc->b[3] = sdc->s[3] >> 8;
-
-      sdc->f[0] = color.f[0];
-      sdc->f[1] = color.f[1];
-      sdc->f[2] = color.f[2];
-      sdc->f[3] = color.f[3];
-   } else {
-      float *sdc = brw_state_batch(brw, 4 * 4, 32, sdc_offset);
-      memcpy(sdc, color.f, 4 * 4);
-   }
-}
-
-/**
- * Sets the sampler state for a single unit based off of the sampler key
- * entry.
- */
-static void
-brw_update_sampler_state(struct brw_context *brw,
-                         GLenum target, bool tex_cube_map_seamless,
-                         GLfloat tex_unit_lod_bias,
-                         mesa_format format, GLenum base_format,
-                         const struct gl_texture_object *texObj,
-                         const struct gl_sampler_object *sampler,
-                         uint32_t *sampler_state,
-                         uint32_t batch_offset_for_sampler_state)
-{
-   unsigned min_filter, mag_filter, mip_filter;
-
-   /* Select min and mip filters. */
-   switch (sampler->MinFilter) {
-   case GL_NEAREST:
-      min_filter = BRW_MAPFILTER_NEAREST;
-      mip_filter = BRW_MIPFILTER_NONE;
-      break;
-   case GL_LINEAR:
-      min_filter = BRW_MAPFILTER_LINEAR;
-      mip_filter = BRW_MIPFILTER_NONE;
-      break;
-   case GL_NEAREST_MIPMAP_NEAREST:
-      min_filter = BRW_MAPFILTER_NEAREST;
-      mip_filter = BRW_MIPFILTER_NEAREST;
-      break;
-   case GL_LINEAR_MIPMAP_NEAREST:
-      min_filter = BRW_MAPFILTER_LINEAR;
-      mip_filter = BRW_MIPFILTER_NEAREST;
-      break;
-   case GL_NEAREST_MIPMAP_LINEAR:
-      min_filter = BRW_MAPFILTER_NEAREST;
-      mip_filter = BRW_MIPFILTER_LINEAR;
-      break;
-   case GL_LINEAR_MIPMAP_LINEAR:
-      min_filter = BRW_MAPFILTER_LINEAR;
-      mip_filter = BRW_MIPFILTER_LINEAR;
-      break;
-   default:
-      unreachable("not reached");
-   }
-
-   /* Select mag filter. */
-   if (sampler->MagFilter == GL_LINEAR)
-      mag_filter = BRW_MAPFILTER_LINEAR;
-   else
-      mag_filter = BRW_MAPFILTER_NEAREST;
-
-   /* Enable anisotropic filtering if desired. */
-   unsigned max_anisotropy = BRW_ANISORATIO_2;
-   if (sampler->MaxAnisotropy > 1.0f) {
-      if (min_filter == BRW_MAPFILTER_LINEAR)
-         min_filter = BRW_MAPFILTER_ANISOTROPIC;
-      if (mag_filter == BRW_MAPFILTER_LINEAR)
-         mag_filter = BRW_MAPFILTER_ANISOTROPIC;
-
-      if (sampler->MaxAnisotropy > 2.0f) {
-        max_anisotropy =
-            MIN2((sampler->MaxAnisotropy - 2) / 2, BRW_ANISORATIO_16);
-      }
-   }
-
-   /* Set address rounding bits if not using nearest filtering. */
-   unsigned address_rounding = 0;
-   if (min_filter != BRW_MAPFILTER_NEAREST) {
-      address_rounding |= BRW_ADDRESS_ROUNDING_ENABLE_U_MIN |
-                          BRW_ADDRESS_ROUNDING_ENABLE_V_MIN |
-                          BRW_ADDRESS_ROUNDING_ENABLE_R_MIN;
-   }
-   if (mag_filter != BRW_MAPFILTER_NEAREST) {
-      address_rounding |= BRW_ADDRESS_ROUNDING_ENABLE_U_MAG |
-                          BRW_ADDRESS_ROUNDING_ENABLE_V_MAG |
-                          BRW_ADDRESS_ROUNDING_ENABLE_R_MAG;
-   }
-
-   bool either_nearest =
-      sampler->MinFilter == GL_NEAREST || sampler->MagFilter == GL_NEAREST;
-   unsigned wrap_s = translate_wrap_mode(brw, sampler->WrapS, either_nearest);
-   unsigned wrap_t = translate_wrap_mode(brw, sampler->WrapT, either_nearest);
-   unsigned wrap_r = translate_wrap_mode(brw, sampler->WrapR, either_nearest);
-
-   if (target == GL_TEXTURE_CUBE_MAP ||
-       target == GL_TEXTURE_CUBE_MAP_ARRAY) {
-      /* Cube maps must use the same wrap mode for all three coordinate
-       * dimensions.  Prior to Haswell, only CUBE and CLAMP are valid.
-       *
-       * Ivybridge and Baytrail seem to have problems with CUBE mode and
-       * integer formats.  Fall back to CLAMP for now.
-       */
-      if ((tex_cube_map_seamless || sampler->CubeMapSeamless) &&
-          !(brw->gen == 7 && !brw->is_haswell && texObj->_IsIntegerFormat)) {
-        wrap_s = BRW_TEXCOORDMODE_CUBE;
-        wrap_t = BRW_TEXCOORDMODE_CUBE;
-        wrap_r = BRW_TEXCOORDMODE_CUBE;
-      } else {
-        wrap_s = BRW_TEXCOORDMODE_CLAMP;
-        wrap_t = BRW_TEXCOORDMODE_CLAMP;
-        wrap_r = BRW_TEXCOORDMODE_CLAMP;
-      }
-   } else if (target == GL_TEXTURE_1D) {
-      /* There's a bug in 1D texture sampling - it actually pays
-       * attention to the wrap_t value, though it should not.
-       * Override the wrap_t value here to GL_REPEAT to keep
-       * any nonexistent border pixels from floating in.
-       */
-      wrap_t = BRW_TEXCOORDMODE_WRAP;
-   }
-
-   /* Set shadow function. */
-   unsigned shadow_function = 0;
-   if (sampler->CompareMode == GL_COMPARE_R_TO_TEXTURE_ARB) {
-      shadow_function =
-        intel_translate_shadow_compare_func(sampler->CompareFunc);
-   }
-
-   const int lod_bits = brw->gen >= 7 ? 8 : 6;
-   const float hw_max_lod = brw->gen >= 7 ? 14 : 13;
-   const unsigned base_level =
-      U_FIXED(CLAMP(texObj->MinLevel + texObj->BaseLevel, 0, hw_max_lod), 1);
-   const unsigned min_lod =
-      U_FIXED(CLAMP(sampler->MinLod, 0, hw_max_lod), lod_bits);
-   const unsigned max_lod =
-      U_FIXED(CLAMP(sampler->MaxLod, 0, hw_max_lod), lod_bits);
-   const int lod_bias =
-      S_FIXED(CLAMP(tex_unit_lod_bias + sampler->LodBias, -16, 15), lod_bits);
-
-   /* Upload the border color if necessary.  If not, just point it at
-    * offset 0 (the start of the batch) - the color should be ignored,
-    * but that address won't fault in case something reads it anyway.
-    */
-   uint32_t border_color_offset = 0;
-   if (wrap_mode_needs_border_color(wrap_s) ||
-       wrap_mode_needs_border_color(wrap_t) ||
-       wrap_mode_needs_border_color(wrap_r)) {
-      upload_default_color(brw, sampler, format, base_format,
-                           texObj->_IsIntegerFormat, texObj->StencilSampling,
-                           &border_color_offset);
-   }
-
-   const bool non_normalized_coords = target == GL_TEXTURE_RECTANGLE;
-
-   brw_emit_sampler_state(brw,
-                          sampler_state,
-                          batch_offset_for_sampler_state,
-                          min_filter, mag_filter, mip_filter,
-                          max_anisotropy,
-                          address_rounding,
-                          wrap_s, wrap_t, wrap_r,
-                          base_level, min_lod, max_lod, lod_bias,
-                          shadow_function,
-                          non_normalized_coords,
-                          border_color_offset);
-}
-
-static void
-update_sampler_state(struct brw_context *brw,
-                     int unit,
-                     uint32_t *sampler_state,
-                     uint32_t batch_offset_for_sampler_state)
-{
-   struct gl_context *ctx = &brw->ctx;
-   const struct gl_texture_unit *texUnit = &ctx->Texture.Unit[unit];
-   const struct gl_texture_object *texObj = texUnit->_Current;
-   const struct gl_sampler_object *sampler = _mesa_get_samplerobj(ctx, unit);
-
-   /* These don't use samplers at all. */
-   if (texObj->Target == GL_TEXTURE_BUFFER)
-      return;
-
-   struct gl_texture_image *firstImage = texObj->Image[0][texObj->BaseLevel];
-   brw_update_sampler_state(brw, texObj->Target, ctx->Texture.CubeMapSeamless,
-                            texUnit->LodBias,
-                            firstImage->TexFormat, firstImage->_BaseFormat,
-                            texObj, sampler,
-                            sampler_state, batch_offset_for_sampler_state);
-}
-
-static void
-brw_upload_sampler_state_table(struct brw_context *brw,
-                               struct gl_program *prog,
-                               struct brw_stage_state *stage_state)
-{
-   struct gl_context *ctx = &brw->ctx;
-   uint32_t sampler_count = stage_state->sampler_count;
-
-   GLbitfield SamplersUsed = prog->SamplersUsed;
-
-   if (sampler_count == 0)
-      return;
-
-   /* SAMPLER_STATE is 4 DWords on all platforms. */
-   const int dwords = 4;
-   const int size_in_bytes = dwords * sizeof(uint32_t);
-
-   uint32_t *sampler_state = brw_state_batch(brw,
-                                             sampler_count * size_in_bytes,
-                                             32, &stage_state->sampler_offset);
-   memset(sampler_state, 0, sampler_count * size_in_bytes);
-
-   uint32_t batch_offset_for_sampler_state = stage_state->sampler_offset;
-
-   for (unsigned s = 0; s < sampler_count; s++) {
-      if (SamplersUsed & (1 << s)) {
-         const unsigned unit = prog->SamplerUnits[s];
-         if (ctx->Texture.Unit[unit]._Current) {
-            update_sampler_state(brw, unit, sampler_state,
-                                     batch_offset_for_sampler_state);
-         }
-      }
-
-      sampler_state += dwords;
-      batch_offset_for_sampler_state += size_in_bytes;
-   }
-
-   if (brw->gen >= 7 && stage_state->stage != MESA_SHADER_COMPUTE) {
-      /* Emit a 3DSTATE_SAMPLER_STATE_POINTERS_XS packet. */
-      gen7_emit_sampler_state_pointers_xs(brw, stage_state);
-   } else {
-      /* Flag that the sampler state table pointer has changed; later atoms
-       * will handle it.
-       */
-      brw->ctx.NewDriverState |= BRW_NEW_SAMPLER_STATE_TABLE;
-   }
-}