i965: Fix num_uniforms count for scalar GS.
authorKenneth Graunke <kenneth@whitecape.org>
Fri, 13 Nov 2015 21:29:16 +0000 (13:29 -0800)
committerKenneth Graunke <kenneth@whitecape.org>
Sun, 22 Nov 2015 08:03:21 +0000 (00:03 -0800)
I noticed that brw_vs.c does this.

I believe the point is that nir->num_uniforms is either counted in
scalar components (in scalar mode), or vec4 slots (in vector mode).
But we want param_count to be in scalar components regardless, so
we have to scale up in vector mode.

We don't have to scale up in scalar mode, though.

Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Jason Ekstrand <jason.ekstrand@intel.com>
src/mesa/drivers/dri/i965/brw_gs.c

index ad5b242a3ab454927dce66aaf7f037ddbb4069c7..149b43ba05552a4315651ae3d1b0641cb97dc577 100644 (file)
@@ -75,7 +75,9 @@ brw_codegen_gs_prog(struct brw_context *brw,
     * every uniform is a float which gets padded to the size of a vec4.
     */
    struct gl_shader *gs = prog->_LinkedShaders[MESA_SHADER_GEOMETRY];
-   int param_count = gp->program.Base.nir->num_uniforms * 4;
+   int param_count = gp->program.Base.nir->num_uniforms;
+   if (!compiler->scalar_stage[MESA_SHADER_GEOMETRY])
+      param_count *= 4;
 
    prog_data.base.base.param =
       rzalloc_array(NULL, const gl_constant_value *, param_count);