diff options
author | Alejandro Piñeiro <[email protected]> | 2018-09-04 13:02:57 +0200 |
---|---|---|
committer | Alejandro Piñeiro <[email protected]> | 2018-09-05 17:02:28 +0200 |
commit | 4e1f8d82c2176be273a8d49f804f5b817252facc (patch) | |
tree | a666e039a5f4c4602fd727825633f20a24fc2dce | |
parent | 8969777686307c5649916d0e2d1024ae1e1af1ce (diff) |
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]>
-rw-r--r-- | src/intel/compiler/brw_fs_nir.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/intel/compiler/brw_fs_nir.cpp b/src/intel/compiler/brw_fs_nir.cpp index 2fef050f81a..e5215bad942 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"); } |