svga: skip vertex attribute instruction with zero usage_mask
In emit_input_declarations(), we are skipping declarations for those registers that are not being used. But in emit_vertex_attrib_instructions(), we are still emitting instructions to tweak the vertex attributes even if they are not being used. This causes an assert in the backend because an input register is not declared in the shader. This patch fixes the problem by skipping the instruction if the vertex attribute is not being used. Changes in this patch is originated from the code snippet from Jose as suggested in bug 1530161. Tested with piglit, Heaven, Turbine, glretrace. Reviewed-by: Jose Fonseca <jfonseca@vmware.com> Reviewed-by: Brian Paul <brianp@vmware.com>
This commit is contained in:
committed by
Brian Paul
parent
b59fad8478
commit
b074a5b02d
@@ -6170,6 +6170,11 @@ emit_vertex_attrib_instructions(struct svga_shader_emitter_v10 *emit)
|
||||
|
||||
while (adjust_mask) {
|
||||
unsigned index = u_bit_scan(&adjust_mask);
|
||||
|
||||
/* skip the instruction if this vertex attribute is not being used */
|
||||
if (emit->info.input_usage_mask[index] == 0)
|
||||
continue;
|
||||
|
||||
unsigned tmp = emit->vs.adjusted_input[index];
|
||||
struct tgsi_full_src_register input_src =
|
||||
make_src_reg(TGSI_FILE_INPUT, index);
|
||||
|
||||
Reference in New Issue
Block a user