Module: Mesa Branch: master Commit: 4e1f8d82c2176be273a8d49f804f5b817252facc URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=4e1f8d82c2176be273a8d49f804f5b817252facc
Author: Alejandro PiƱeiro <[email protected]> Date: Tue Sep 4 13:02:57 2018 +0200 i965/fs: include multisamplers on image_intrinsic_coord_components This is the second patch needed to fix the following piglit tests: tests/spec/arb_gl_spirv/linker/uniform/multisampler.shader_test tests/spec/arb_gl_spirv/linker/uniform/multisampler-array.shader_test Although in this case it doesn't affect so many borrowed tests, as there aren't too many tests using multisamplers on Intel. It is worth to note that this patch is also needed when those tests are run on GLSL mode (using the --glsl option). Although most Intel drivers would not be able to run/execute tests using multisamplers, as GL_MAX_IMAGE_SAMPLES is zero, technically those tests are expected to link correctly, so linking tests should pass. Reviewed-by: Jason Ekstrand <[email protected]> --- src/intel/compiler/brw_fs_nir.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/intel/compiler/brw_fs_nir.cpp b/src/intel/compiler/brw_fs_nir.cpp index 2fef050f81..e5215bad94 100644 --- a/src/intel/compiler/brw_fs_nir.cpp +++ b/src/intel/compiler/brw_fs_nir.cpp @@ -3815,6 +3815,8 @@ image_intrinsic_coord_components(nir_intrinsic_instr *instr) return 3; case GLSL_SAMPLER_DIM_BUF: return 1; + case GLSL_SAMPLER_DIM_MS: + return 2 + nir_intrinsic_image_array(instr); default: unreachable("Invalid image dimension"); } _______________________________________________ mesa-commit mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-commit
