diff options
| author | Jordan Justen <jordan.l.justen@intel.com> | 2016-05-28 17:57:31 -0700 |
|---|---|---|
| committer | Jordan Justen <jordan.l.justen@intel.com> | 2016-05-29 09:59:55 -0700 |
| commit | 7398a32c501ed7fedb5619ee7505f9070551d4bd (patch) | |
| tree | 2639d5cfd78f6e5aa7c955ba48bba4768704e79c /src/mesa/drivers/dri/i965/brw_cs.c | |
| parent | 160063b110d50d528217492308bbe353af2186e8 (diff) | |
| download | external_mesa3d-7398a32c501ed7fedb5619ee7505f9070551d4bd.zip external_mesa3d-7398a32c501ed7fedb5619ee7505f9070551d4bd.tar.gz external_mesa3d-7398a32c501ed7fedb5619ee7505f9070551d4bd.tar.bz2 | |
i965: Shrink stage_prog_data param array length
It appears we were over-allocating these arrays.
Previously we would use nir->num_uniforms directly for scalar
programs, and multiply it by 4 for vec4 programs.
Instead we should have been dividing by 4 in both cases to convert
from bytes to a gl_constant_value count. The size of gl_constant_value
is 4 bytes.
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com>
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
Diffstat (limited to 'src/mesa/drivers/dri/i965/brw_cs.c')
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_cs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_cs.c b/src/mesa/drivers/dri/i965/brw_cs.c index 0ab9ebd..a9cbde9 100644 --- a/src/mesa/drivers/dri/i965/brw_cs.c +++ b/src/mesa/drivers/dri/i965/brw_cs.c @@ -91,7 +91,7 @@ brw_codegen_cs_prog(struct brw_context *brw, * prog_data associated with the compiled program, and which will be freed * by the state cache. */ - int param_count = cp->program.Base.nir->num_uniforms; + int param_count = cp->program.Base.nir->num_uniforms / 4; /* The backend also sometimes adds params for texture size. */ param_count += 2 * ctx->Const.Program[MESA_SHADER_COMPUTE].MaxTextureImageUnits; |