On 10/21/2011 12:49 PM, Ian Romanick wrote:
From: Ian Romanick<ian.d.roman...@intel.com>

Setting this flag prevents declarations of uniforms from being removed
from the IR.  Since the IR is directly used by several API functions
that query uniforms in shaders, uniform declarations cannot be removed
after the locations have been set.  However, it should still be safe
to reorder the declarations (this is not tested).

Signed-off-by: Ian Romanick<ian.d.roman...@intel.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=41980
Cc: Brian Paul<bri...@vmware.com>
Cc: Bryan Cain<bryanca...@gmail.com>
Cc: Vinson Lee<v...@vmware.com>
Cc: José Fonseca<jfons...@vmware.com>
Cc: Kenneth Graunke<kenn...@whitecape.org>

I applied these two patches to my tree but I'm still seeing the GL_INVALID_OPERATION error in glUniform() calls that I mentioned in the bug report (param->DataType = 0).

The viewperf test I'm using has a fairly complex vertex shader with an array of structures of lighting info containing a "vec3 position" field.

The test calls glGetUniform(shader, "LW_LightSource1[0].position"). When glUniform3f() is called to set that uniform, the GL_INVALID_OPERATION errro is raised.

I'll try a bisection to see where this started failing.

-Brian
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to