diff options
| author | Jason Ekstrand <[email protected]> | 2018-07-23 09:41:26 -0700 |
|---|---|---|
| committer | Jason Ekstrand <[email protected]> | 2018-07-23 15:28:17 -0700 |
| commit | 820d5e51b7060f02d6c12fbb1c349111022ff37a (patch) | |
| tree | 6da91c27fc93007d2417c64a883de0dcb75835b4 /src/mesa/drivers/dri/i965/brw_vs.c | |
| parent | 62024fa775058013a5a75f576f1129239c95de11 (diff) | |
intel/compiler: Account for built-in uniforms in analyze_ubo_ranges
The original pass only looked for load_uniform intrinsics but there are
a number of other places that could end up loading a push constant. One
obvious omission was images which always implicitly use a push constant.
Legacy VS clip planes also get pushed into the shader. This fixes some
new Vulkan CTS tests that test random combinations of bindings and, in
particular, test lots of UBOs and images together.
Cc: [email protected]
Cc: Kenneth Graunke <[email protected]>
Diffstat (limited to 'src/mesa/drivers/dri/i965/brw_vs.c')
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_vs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_vs.c b/src/mesa/drivers/dri/i965/brw_vs.c index f518649e751..69c0046bbb9 100644 --- a/src/mesa/drivers/dri/i965/brw_vs.c +++ b/src/mesa/drivers/dri/i965/brw_vs.c @@ -181,7 +181,7 @@ brw_codegen_vs_prog(struct brw_context *brw, brw_nir_setup_glsl_uniforms(mem_ctx, vp->program.nir, &vp->program, &prog_data.base.base, compiler->scalar_stage[MESA_SHADER_VERTEX]); - brw_nir_analyze_ubo_ranges(compiler, vp->program.nir, + brw_nir_analyze_ubo_ranges(compiler, vp->program.nir, key, prog_data.base.base.ubo_ranges); } else { brw_nir_setup_arb_uniforms(mem_ctx, vp->program.nir, &vp->program, |