iris: Leave a comment about why Broadwell images are broken
authorKenneth Graunke <kenneth@whitecape.org>
Mon, 31 Dec 2018 17:19:07 +0000 (09:19 -0800)
committerKenneth Graunke <kenneth@whitecape.org>
Thu, 21 Feb 2019 18:26:11 +0000 (10:26 -0800)
There are a variety of ways to fix this, many of which are simple, but
I could use some advice on which ones other people prefer, and so we'll
punt until after the holidays.

src/gallium/drivers/iris/iris_program.c

index 47a4b635f99a3961e73571fdc748e8f432879c09..499af26da9de30579fc2863313da20265c3019a6 100644 (file)
@@ -668,6 +668,10 @@ iris_setup_uniforms(const struct brw_compiler *compiler,
             nir_deref_instr *deref = nir_src_as_deref(intrin->src[0]);
             nir_variable *var = nir_deref_instr_get_variable(deref);
 
+            /* XXX: var->data.binding is not set properly.  We need to run
+             * some form of gl_nir_lower_samplers_as_deref() to get it.
+             * This breaks tests which use more than one image.
+             */
             if (img_idx[var->data.binding] == -1) {
                /* GL only allows arrays of arrays of images. */
                assert(glsl_type_is_image(glsl_without_array(var->type)));