<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Wed, Sep 5, 2018 at 9:11 AM Alejandro Piñeiro <<a href="mailto:apinheiro@igalia.com">apinheiro@igalia.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This is the second patch needed to fix the following piglit tests:<br>
<br>
   tests/spec/arb_gl_spirv/linker/uniform/multisampler.shader_test<br>
   tests/spec/arb_gl_spirv/linker/uniform/multisampler-array.shader_test<br>
<br>
Although in this case it doesn't affect so many borrowed tests, as<br>
there aren't too many tests using multisamplers on Intel.<br>
<br>
It is worth to note that this patch is also needed when those tests<br>
are run on GLSL mode (using the --glsl option). Although most Intel<br>
drivers would not be able to run/execute tests using multisamplers, as<br>
GL_MAX_IMAGE_SAMPLES is zero, technically those tests are expected to<br>
link correctly, so linking tests should pass.<br></blockquote><div><br></div><div>That's a bit aggrivating... Oh, well.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
---<br>
 src/intel/compiler/brw_fs_nir.cpp | 2 ++<br>
 1 file changed, 2 insertions(+)<br>
<br>
diff --git a/src/intel/compiler/brw_fs_nir.cpp b/src/intel/compiler/brw_fs_nir.cpp<br>
index 2fef050f81a..eef3453672c 100644<br>
--- a/src/intel/compiler/brw_fs_nir.cpp<br>
+++ b/src/intel/compiler/brw_fs_nir.cpp<br>
@@ -3815,6 +3815,8 @@ image_intrinsic_coord_components(nir_intrinsic_instr *instr)<br>
       return 3;<br>
    case GLSL_SAMPLER_DIM_BUF:<br>
       return 1;<br>
+   case GLSL_SAMPLER_DIM_MS:<br>
+      return 2;<br></blockquote><div><br></div><div>This should be 2 + nir_intrinsic_image_array(instr) because multisampled images can also be arrays.  With that fixed,</div><div><br></div><div>Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br></div></div></div>