g3dvl: remove unused vertex shader inputs
authorMarek Olšák <maraeo@gmail.com>
Fri, 15 Jul 2011 18:16:20 +0000 (20:16 +0200)
committerMarek Olšák <maraeo@gmail.com>
Wed, 20 Jul 2011 20:55:24 +0000 (22:55 +0200)
See also comments in the code.

src/gallium/auxiliary/vl/vl_idct.c
src/gallium/auxiliary/vl/vl_mc.c

index 645d06a0925d14072433543a272dc340b3c9e2bf..ad78614539262eef20cbca10dbf9e5731b1f274d 100644 (file)
@@ -143,7 +143,7 @@ static void *
 create_mismatch_vert_shader(struct vl_idct *idct)
 {
    struct ureg_program *shader;
-   struct ureg_src vrect, vpos;
+   struct ureg_src vpos;
    struct ureg_src scale;
    struct ureg_dst t_tex;
    struct ureg_dst o_vpos, o_addr[2];
@@ -152,7 +152,6 @@ create_mismatch_vert_shader(struct vl_idct *idct)
    if (!shader)
       return NULL;
 
-   vrect = ureg_DECL_vs_input(shader, VS_I_RECT);
    vpos = ureg_DECL_vs_input(shader, VS_I_VPOS);
 
    t_tex = ureg_DECL_temporary(shader);
index add367e3ac6b1e46b18b0e526220833b120c63c0..81a05b539f3921b5d08bbebe7117d3b9ba2994c5 100644 (file)
@@ -103,7 +103,7 @@ create_ref_vert_shader(struct vl_mc *r)
 {
    struct ureg_program *shader;
    struct ureg_src mv_scale;
-   struct ureg_src vrect, vmv[2];
+   struct ureg_src vmv[2];
    struct ureg_dst t_vpos;
    struct ureg_dst o_vpos, o_vmv[2];
    unsigned i;
@@ -112,7 +112,6 @@ create_ref_vert_shader(struct vl_mc *r)
    if (!shader)
       return NULL;
 
-   vrect = ureg_DECL_vs_input(shader, VS_I_RECT);
    vmv[0] = ureg_DECL_vs_input(shader, VS_I_MV_TOP);
    vmv[1] = ureg_DECL_vs_input(shader, VS_I_MV_BOTTOM);
 
@@ -121,6 +120,8 @@ create_ref_vert_shader(struct vl_mc *r)
       (float)MACROBLOCK_HEIGHT / r->buffer_height)
    );
 
+   /* XXX The position is not written, which may lead to undefined rendering.
+    * XXX This is a serious bug. */
    o_vpos = ureg_DECL_output(shader, TGSI_SEMANTIC_POSITION, VS_O_VPOS);
    o_vmv[0] = ureg_DECL_output(shader, TGSI_SEMANTIC_GENERIC, VS_O_VTOP);
    o_vmv[1] = ureg_DECL_output(shader, TGSI_SEMANTIC_GENERIC, VS_O_VBOTTOM);