r600g: take into account force_add_cf in pops
[mesa.git] / src / gallium / drivers / nvfx / nvfx_vertprog.c
index 3547cad6aebbb5330c8114990f210a5734ab9aa5..a11941f3d51ee89b0f43d098e1e382c451ba5ba1 100644 (file)
@@ -1,15 +1,20 @@
 #include "pipe/p_context.h"
 #include "pipe/p_defines.h"
 #include "pipe/p_state.h"
-#include "util/u_inlines.h"
+#include "util/u_linkage.h"
+#include "util/u_debug.h"
 
 #include "pipe/p_shader_tokens.h"
 #include "tgsi/tgsi_parse.h"
 #include "tgsi/tgsi_dump.h"
 #include "tgsi/tgsi_util.h"
+#include "tgsi/tgsi_ureg.h"
+
+#include "draw/draw_context.h"
 
 #include "nvfx_context.h"
 #include "nvfx_state.h"
+#include "nvfx_resource.h"
 
 /* TODO (at least...):
  *  1. Indexed consts  + ARL
 #include "nv30_vertprog.h"
 #include "nv40_vertprog.h"
 
-#define NVFX_VP_INST_DEST_CLIP(n) ((~0 - 6) + (n))
+struct nvfx_loop_entry
+{
+       unsigned brk_target;
+       unsigned cont_target;
+};
 
 struct nvfx_vpc {
+       struct nvfx_context* nvfx;
+       struct pipe_shader_state pipe;
        struct nvfx_vertex_program *vp;
+       struct tgsi_shader_info* info;
 
        struct nvfx_vertex_program_exec *vpi;
 
        unsigned r_temps;
        unsigned r_temps_discard;
-       struct nvfx_sreg r_result[PIPE_MAX_SHADER_OUTPUTS];
-       struct nvfx_sreg *r_address;
-       struct nvfx_sreg *r_temp;
+       struct nvfx_reg r_result[PIPE_MAX_SHADER_OUTPUTS];
+       struct nvfx_reg *r_address;
+       struct nvfx_reg *r_temp;
+       struct nvfx_reg *r_const;
+       struct nvfx_reg r_0_1;
 
-       struct nvfx_sreg *imm;
+       struct nvfx_reg *imm;
        unsigned nr_imm;
 
        unsigned hpos_idx;
+
+       struct util_dynarray label_relocs;
+       struct util_dynarray loop_stack;
 };
 
-static struct nvfx_sreg
+static struct nvfx_reg
 temp(struct nvfx_vpc *vpc)
 {
        int idx = ffs(~vpc->r_temps) - 1;
@@ -52,22 +69,22 @@ temp(struct nvfx_vpc *vpc)
        if (idx < 0) {
                NOUVEAU_ERR("out of temps!!\n");
                assert(0);
-               return nvfx_sr(NVFXSR_TEMP, 0);
+               return nvfx_reg(NVFXSR_TEMP, 0);
        }
 
        vpc->r_temps |= (1 << idx);
        vpc->r_temps_discard |= (1 << idx);
-       return nvfx_sr(NVFXSR_TEMP, idx);
+       return nvfx_reg(NVFXSR_TEMP, idx);
 }
 
-static INLINE void
+static inline void
 release_temps(struct nvfx_vpc *vpc)
 {
        vpc->r_temps &= ~vpc->r_temps_discard;
        vpc->r_temps_discard = 0;
 }
 
-static struct nvfx_sreg
+static struct nvfx_reg
 constant(struct nvfx_vpc *vpc, int pipe, float x, float y, float z, float w)
 {
        struct nvfx_vertex_program *vp = vpc->vp;
@@ -77,7 +94,7 @@ constant(struct nvfx_vpc *vpc, int pipe, float x, float y, float z, float w)
        if (pipe >= 0) {
                for (idx = 0; idx < vp->nr_consts; idx++) {
                        if (vp->consts[idx].index == pipe)
-                               return nvfx_sr(NVFXSR_CONST, idx);
+                               return nvfx_reg(NVFXSR_CONST, idx);
                }
        }
 
@@ -90,35 +107,36 @@ constant(struct nvfx_vpc *vpc, int pipe, float x, float y, float z, float w)
        vpd->value[1] = y;
        vpd->value[2] = z;
        vpd->value[3] = w;
-       return nvfx_sr(NVFXSR_CONST, idx);
+       return nvfx_reg(NVFXSR_CONST, idx);
 }
 
-#define arith(cc,s,o,d,m,s0,s1,s2) \
-       nvfx_vp_arith(nvfx, (cc), NVFX_VP_INST_SLOT_##s, NVFX_VP_INST_##s##_OP_##o, (d), (m), (s0), (s1), (s2))
+#define arith(s,t,o,d,m,s0,s1,s2) \
+       nvfx_insn((s), (NVFX_VP_INST_SLOT_##t << 7) | NVFX_VP_INST_##t##_OP_##o, -1, (d), (m), (s0), (s1), (s2))
 
 static void
-emit_src(struct nvfx_context* nvfx, struct nvfx_vpc *vpc, uint32_t *hw, int pos, struct nvfx_sreg src)
+emit_src(struct nvfx_context* nvfx, struct nvfx_vpc *vpc, uint32_t *hw, int pos, struct nvfx_src src)
 {
        struct nvfx_vertex_program *vp = vpc->vp;
        uint32_t sr = 0;
+       struct nvfx_relocation reloc;
 
-       switch (src.type) {
+       switch (src.reg.type) {
        case NVFXSR_TEMP:
                sr |= (NVFX_VP(SRC_REG_TYPE_TEMP) << NVFX_VP(SRC_REG_TYPE_SHIFT));
-               sr |= (src.index << NVFX_VP(SRC_TEMP_SRC_SHIFT));
+               sr |= (src.reg.index << NVFX_VP(SRC_TEMP_SRC_SHIFT));
                break;
        case NVFXSR_INPUT:
                sr |= (NVFX_VP(SRC_REG_TYPE_INPUT) <<
                       NVFX_VP(SRC_REG_TYPE_SHIFT));
-               vp->ir |= (1 << src.index);
-               hw[1] |= (src.index << NVFX_VP(INST_INPUT_SRC_SHIFT));
+               vp->ir |= (1 << src.reg.index);
+               hw[1] |= (src.reg.index << NVFX_VP(INST_INPUT_SRC_SHIFT));
                break;
        case NVFXSR_CONST:
                sr |= (NVFX_VP(SRC_REG_TYPE_CONST) <<
                       NVFX_VP(SRC_REG_TYPE_SHIFT));
-               assert(vpc->vpi->const_index == -1 ||
-                      vpc->vpi->const_index == src.index);
-               vpc->vpi->const_index = src.index;
+               reloc.location = vp->nr_insns - 1;
+               reloc.target = src.reg.index;
+               util_dynarray_append(&vp->const_relocs, struct nvfx_relocation, reloc);
                break;
        case NVFXSR_NONE:
                sr |= (NVFX_VP(SRC_REG_TYPE_INPUT) <<
@@ -139,6 +157,18 @@ emit_src(struct nvfx_context* nvfx, struct nvfx_vpc *vpc, uint32_t *hw, int pos,
               (src.swz[2] << NVFX_VP(SRC_SWZ_Z_SHIFT)) |
               (src.swz[3] << NVFX_VP(SRC_SWZ_W_SHIFT)));
 
+       if(src.indirect) {
+               if(src.reg.type == NVFXSR_CONST)
+                       hw[3] |= NVFX_VP(INST_INDEX_CONST);
+               else if(src.reg.type == NVFXSR_INPUT)
+                       hw[0] |= NVFX_VP(INST_INDEX_INPUT);
+               else
+                       assert(0);
+               if(src.indirect_reg)
+                       hw[0] |= NVFX_VP(INST_ADDR_REG_SELECT_1);
+               hw[0] |= src.indirect_swz << NVFX_VP(INST_ADDR_SWZ_SHIFT);
+       }
+
        switch (pos) {
        case 0:
                hw[1] |= ((sr & NVFX_VP(SRC0_HIGH_MASK)) >>
@@ -161,100 +191,67 @@ emit_src(struct nvfx_context* nvfx, struct nvfx_vpc *vpc, uint32_t *hw, int pos,
 }
 
 static void
-emit_dst(struct nvfx_context* nvfx, struct nvfx_vpc *vpc, uint32_t *hw, int slot, struct nvfx_sreg dst)
+emit_dst(struct nvfx_context* nvfx, struct nvfx_vpc *vpc, uint32_t *hw, int slot, struct nvfx_reg dst)
 {
        struct nvfx_vertex_program *vp = vpc->vp;
 
        switch (dst.type) {
+       case NVFXSR_NONE:
+               if(!nvfx->is_nv4x)
+                       hw[0] |= NV30_VP_INST_DEST_TEMP_ID_MASK;
+               else {
+                       hw[3] |= NV40_VP_INST_DEST_MASK;
+                       if (slot == 0)
+                               hw[0] |= NV40_VP_INST_VEC_DEST_TEMP_MASK;
+                       else
+                               hw[3] |= NV40_VP_INST_SCA_DEST_TEMP_MASK;
+               }
+               break;
        case NVFXSR_TEMP:
                if(!nvfx->is_nv4x)
                        hw[0] |= (dst.index << NV30_VP_INST_DEST_TEMP_ID_SHIFT);
                else {
                        hw[3] |= NV40_VP_INST_DEST_MASK;
-                       if (slot == 0) {
-                               hw[0] |= (dst.index <<
-                                         NV40_VP_INST_VEC_DEST_TEMP_SHIFT);
-                       } else {
-                               hw[3] |= (dst.index <<
-                                         NV40_VP_INST_SCA_DEST_TEMP_SHIFT);
-                       }
+                       if (slot == 0)
+                               hw[0] |= (dst.index << NV40_VP_INST_VEC_DEST_TEMP_SHIFT);
+                       else
+                               hw[3] |= (dst.index << NV40_VP_INST_SCA_DEST_TEMP_SHIFT);
                }
                break;
        case NVFXSR_OUTPUT:
                /* TODO: this may be wrong because on nv30 COL0 and BFC0 are swapped */
-               switch (dst.index) {
-               case NVFX_VP_INST_DEST_CLIP(0):
-                       vp->or |= (1 << 6);
-                       vp->clip_ctrl |= NV34TCL_VP_CLIP_PLANES_ENABLE_PLANE0;
-                       dst.index = NVFX_VP(INST_DEST_FOGC);
-                       break;
-               case NVFX_VP_INST_DEST_CLIP(1):
-                       vp->or |= (1 << 7);
-                       vp->clip_ctrl |= NV34TCL_VP_CLIP_PLANES_ENABLE_PLANE1;
-                       dst.index = NVFX_VP(INST_DEST_FOGC);
-                       break;
-               case NVFX_VP_INST_DEST_CLIP(2):
-                       vp->or |= (1 << 8);
-                       vp->clip_ctrl |= NV34TCL_VP_CLIP_PLANES_ENABLE_PLANE2;
-                       dst.index = NVFX_VP(INST_DEST_FOGC);
-                       break;
-               case NVFX_VP_INST_DEST_CLIP(3):
-                       vp->or |= (1 << 9);
-                       vp->clip_ctrl |= NV34TCL_VP_CLIP_PLANES_ENABLE_PLANE3;
-                       dst.index = NVFX_VP(INST_DEST_PSZ);
-                       break;
-               case NVFX_VP_INST_DEST_CLIP(4):
-                       vp->or |= (1 << 10);
-                       vp->clip_ctrl |= NV34TCL_VP_CLIP_PLANES_ENABLE_PLANE4;
-                       dst.index = NVFX_VP(INST_DEST_PSZ);
-                       break;
-               case NVFX_VP_INST_DEST_CLIP(5):
-                       vp->or |= (1 << 11);
-                       vp->clip_ctrl |= NV34TCL_VP_CLIP_PLANES_ENABLE_PLANE5;
-                       dst.index = NVFX_VP(INST_DEST_PSZ);
-                       break;
-               default:
-                       if(!nvfx->is_nv4x) {
-                               switch (dst.index) {
-                               case NV30_VP_INST_DEST_COL0 : vp->or |= (1 << 0); break;
-                               case NV30_VP_INST_DEST_COL1 : vp->or |= (1 << 1); break;
-                               case NV30_VP_INST_DEST_BFC0 : vp->or |= (1 << 2); break;
-                               case NV30_VP_INST_DEST_BFC1 : vp->or |= (1 << 3); break;
-                               case NV30_VP_INST_DEST_FOGC: vp->or |= (1 << 4); break;
-                               case NV30_VP_INST_DEST_PSZ  : vp->or |= (1 << 5); break;
-                               case NV30_VP_INST_DEST_TC(0): vp->or |= (1 << 14); break;
-                               case NV30_VP_INST_DEST_TC(1): vp->or |= (1 << 15); break;
-                               case NV30_VP_INST_DEST_TC(2): vp->or |= (1 << 16); break;
-                               case NV30_VP_INST_DEST_TC(3): vp->or |= (1 << 17); break;
-                               case NV30_VP_INST_DEST_TC(4): vp->or |= (1 << 18); break;
-                               case NV30_VP_INST_DEST_TC(5): vp->or |= (1 << 19); break;
-                               case NV30_VP_INST_DEST_TC(6): vp->or |= (1 << 20); break;
-                               case NV30_VP_INST_DEST_TC(7): vp->or |= (1 << 21); break;
-                               }
-                       } else {
-                               switch (dst.index) {
-                               case NV40_VP_INST_DEST_COL0 : vp->or |= (1 << 0); break;
-                               case NV40_VP_INST_DEST_COL1 : vp->or |= (1 << 1); break;
-                               case NV40_VP_INST_DEST_BFC0 : vp->or |= (1 << 2); break;
-                               case NV40_VP_INST_DEST_BFC1 : vp->or |= (1 << 3); break;
-                               case NV40_VP_INST_DEST_FOGC: vp->or |= (1 << 4); break;
-                               case NV40_VP_INST_DEST_PSZ  : vp->or |= (1 << 5); break;
-                               case NV40_VP_INST_DEST_TC(0): vp->or |= (1 << 14); break;
-                               case NV40_VP_INST_DEST_TC(1): vp->or |= (1 << 15); break;
-                               case NV40_VP_INST_DEST_TC(2): vp->or |= (1 << 16); break;
-                               case NV40_VP_INST_DEST_TC(3): vp->or |= (1 << 17); break;
-                               case NV40_VP_INST_DEST_TC(4): vp->or |= (1 << 18); break;
-                               case NV40_VP_INST_DEST_TC(5): vp->or |= (1 << 19); break;
-                               case NV40_VP_INST_DEST_TC(6): vp->or |= (1 << 20); break;
-                               case NV40_VP_INST_DEST_TC(7): vp->or |= (1 << 21); break;
-                               }
+               if(nvfx->is_nv4x) {
+                       switch (dst.index) {
+                       case NV30_VP_INST_DEST_CLP(0):
+                               dst.index = NVFX_VP(INST_DEST_FOGC);
+                               break;
+                       case NV30_VP_INST_DEST_CLP(1):
+                               dst.index = NVFX_VP(INST_DEST_FOGC);
+                               break;
+                       case NV30_VP_INST_DEST_CLP(2):
+                               dst.index = NVFX_VP(INST_DEST_FOGC);
+                               break;
+                       case NV30_VP_INST_DEST_CLP(3):
+                               dst.index = NVFX_VP(INST_DEST_PSZ);
+                               break;
+                       case NV30_VP_INST_DEST_CLP(4):
+                               dst.index = NVFX_VP(INST_DEST_PSZ);
+                               break;
+                       case NV30_VP_INST_DEST_CLP(5):
+                               dst.index = NVFX_VP(INST_DEST_PSZ);
+                               break;
+                       case NV40_VP_INST_DEST_COL0 : vp->or |= (1 << 0); break;
+                       case NV40_VP_INST_DEST_COL1 : vp->or |= (1 << 1); break;
+                       case NV40_VP_INST_DEST_BFC0 : vp->or |= (1 << 2); break;
+                       case NV40_VP_INST_DEST_BFC1 : vp->or |= (1 << 3); break;
+                       case NV40_VP_INST_DEST_FOGC: vp->or |= (1 << 4); break;
+                       case NV40_VP_INST_DEST_PSZ  : vp->or |= (1 << 5); break;
                        }
-                       break;
                }
 
                if(!nvfx->is_nv4x) {
                        hw[3] |= (dst.index << NV30_VP_INST_DEST_SHIFT);
-                       hw[0] |= NV30_VP_INST_VEC_DEST_TEMP_MASK | (1<<20);
+                       hw[0] |= NV30_VP_INST_VEC_DEST_TEMP_MASK;
 
                        /*XXX: no way this is entirely correct, someone needs to
                         *     figure out what exactly it is.
@@ -264,7 +261,7 @@ emit_dst(struct nvfx_context* nvfx, struct nvfx_vpc *vpc, uint32_t *hw, int slot
                        hw[3] |= (dst.index << NV40_VP_INST_DEST_SHIFT);
                        if (slot == 0) {
                                hw[0] |= NV40_VP_INST_VEC_RESULT;
-                               hw[0] |= NV40_VP_INST_VEC_DEST_TEMP_MASK | (1<<20);
+                               hw[0] |= NV40_VP_INST_VEC_DEST_TEMP_MASK;
                        } else {
                                hw[3] |= NV40_VP_INST_SCA_RESULT;
                                hw[3] |= NV40_VP_INST_SCA_DEST_TEMP_MASK;
@@ -277,80 +274,99 @@ emit_dst(struct nvfx_context* nvfx, struct nvfx_vpc *vpc, uint32_t *hw, int slot
 }
 
 static void
-nvfx_vp_arith(struct nvfx_context* nvfx, struct nvfx_vpc *vpc, int slot, int op,
-             struct nvfx_sreg dst, int mask,
-             struct nvfx_sreg s0, struct nvfx_sreg s1,
-             struct nvfx_sreg s2)
+nvfx_vp_emit(struct nvfx_vpc *vpc, struct nvfx_insn insn)
 {
+       struct nvfx_context* nvfx = vpc->nvfx;
        struct nvfx_vertex_program *vp = vpc->vp;
+       unsigned slot = insn.op >> 7;
+       unsigned op = insn.op & 0x7f;
        uint32_t *hw;
 
        vp->insns = realloc(vp->insns, ++vp->nr_insns * sizeof(*vpc->vpi));
        vpc->vpi = &vp->insns[vp->nr_insns - 1];
        memset(vpc->vpi, 0, sizeof(*vpc->vpi));
-       vpc->vpi->const_index = -1;
 
        hw = vpc->vpi->data;
 
-       hw[0] |= (NVFX_COND_TR << NVFX_VP(INST_COND_SHIFT));
-       hw[0] |= ((0 << NVFX_VP(INST_COND_SWZ_X_SHIFT)) |
-                 (1 << NVFX_VP(INST_COND_SWZ_Y_SHIFT)) |
-                 (2 << NVFX_VP(INST_COND_SWZ_Z_SHIFT)) |
-                 (3 << NVFX_VP(INST_COND_SWZ_W_SHIFT)));
+       hw[0] |= (insn.cc_test << NVFX_VP(INST_COND_SHIFT));
+       hw[0] |= ((insn.cc_swz[0] << NVFX_VP(INST_COND_SWZ_X_SHIFT)) |
+                 (insn.cc_swz[1] << NVFX_VP(INST_COND_SWZ_Y_SHIFT)) |
+                 (insn.cc_swz[2] << NVFX_VP(INST_COND_SWZ_Z_SHIFT)) |
+                 (insn.cc_swz[3] << NVFX_VP(INST_COND_SWZ_W_SHIFT)));
+       if(insn.cc_update)
+               hw[0] |= NVFX_VP(INST_COND_UPDATE_ENABLE);
+
+       if(insn.sat)
+       {
+               assert(nvfx->use_nv4x);
+               if(nvfx->use_nv4x)
+                       hw[0] |= NV40_VP_INST_SATURATE;
+       }
 
        if(!nvfx->is_nv4x) {
-               hw[1] |= (op << NV30_VP_INST_VEC_OPCODE_SHIFT);
+               if(slot == 0)
+                       hw[1] |= (op << NV30_VP_INST_VEC_OPCODE_SHIFT);
+               else
+               {
+                       hw[0] |= ((op >> 4) << NV30_VP_INST_SCA_OPCODEH_SHIFT);
+                       hw[1] |= ((op & 0xf) << NV30_VP_INST_SCA_OPCODEL_SHIFT);
+               }
 //             hw[3] |= NVFX_VP(INST_SCA_DEST_TEMP_MASK);
 //             hw[3] |= (mask << NVFX_VP(INST_VEC_WRITEMASK_SHIFT));
 
-               if (dst.type == NVFXSR_OUTPUT) {
+               if (insn.dst.type == NVFXSR_OUTPUT) {
                        if (slot)
-                               hw[3] |= (mask << NV30_VP_INST_SDEST_WRITEMASK_SHIFT);
+                               hw[3] |= (insn.mask << NV30_VP_INST_SDEST_WRITEMASK_SHIFT);
                        else
-                               hw[3] |= (mask << NV30_VP_INST_VDEST_WRITEMASK_SHIFT);
+                               hw[3] |= (insn.mask << NV30_VP_INST_VDEST_WRITEMASK_SHIFT);
                } else {
                        if (slot)
-                               hw[3] |= (mask << NV30_VP_INST_STEMP_WRITEMASK_SHIFT);
+                               hw[3] |= (insn.mask << NV30_VP_INST_STEMP_WRITEMASK_SHIFT);
                        else
-                               hw[3] |= (mask << NV30_VP_INST_VTEMP_WRITEMASK_SHIFT);
+                               hw[3] |= (insn.mask << NV30_VP_INST_VTEMP_WRITEMASK_SHIFT);
                }
         } else {
                if (slot == 0) {
                        hw[1] |= (op << NV40_VP_INST_VEC_OPCODE_SHIFT);
                        hw[3] |= NV40_VP_INST_SCA_DEST_TEMP_MASK;
-                       hw[3] |= (mask << NV40_VP_INST_VEC_WRITEMASK_SHIFT);
+                       hw[3] |= (insn.mask << NV40_VP_INST_VEC_WRITEMASK_SHIFT);
            } else {
                        hw[1] |= (op << NV40_VP_INST_SCA_OPCODE_SHIFT);
-                       hw[0] |= (NV40_VP_INST_VEC_DEST_TEMP_MASK | (1 << 20));
-                       hw[3] |= (mask << NV40_VP_INST_SCA_WRITEMASK_SHIFT);
+                       hw[0] |= NV40_VP_INST_VEC_DEST_TEMP_MASK ;
+                       hw[3] |= (insn.mask << NV40_VP_INST_SCA_WRITEMASK_SHIFT);
                }
        }
 
-       emit_dst(nvfx, vpc, hw, slot, dst);
-       emit_src(nvfx, vpc, hw, 0, s0);
-       emit_src(nvfx, vpc, hw, 1, s1);
-       emit_src(nvfx, vpc, hw, 2, s2);
+       emit_dst(nvfx, vpc, hw, slot, insn.dst);
+       emit_src(nvfx, vpc, hw, 0, insn.src[0]);
+       emit_src(nvfx, vpc, hw, 1, insn.src[1]);
+       emit_src(nvfx, vpc, hw, 2, insn.src[2]);
+
+//     if(insn.src[0].indirect || op == NVFX_VP_INST_VEC_OP_ARL)
+//             hw[3] |= NV40_VP_INST_SCA_RESULT;
 }
 
-static INLINE struct nvfx_sreg
+static inline struct nvfx_src
 tgsi_src(struct nvfx_vpc *vpc, const struct tgsi_full_src_register *fsrc) {
-       struct nvfx_sreg src;
+       struct nvfx_src src;
 
        switch (fsrc->Register.File) {
        case TGSI_FILE_INPUT:
-               src = nvfx_sr(NVFXSR_INPUT, fsrc->Register.Index);
+               src.reg = nvfx_reg(NVFXSR_INPUT, fsrc->Register.Index);
                break;
        case TGSI_FILE_CONSTANT:
-               src = constant(vpc, fsrc->Register.Index, 0, 0, 0, 0);
+               src.reg = vpc->r_const[fsrc->Register.Index];
                break;
        case TGSI_FILE_IMMEDIATE:
-               src = vpc->imm[fsrc->Register.Index];
+               src.reg = vpc->imm[fsrc->Register.Index];
                break;
        case TGSI_FILE_TEMPORARY:
-               src = vpc->r_temp[fsrc->Register.Index];
+               src.reg = vpc->r_temp[fsrc->Register.Index];
                break;
        default:
                NOUVEAU_ERR("bad src file\n");
+               src.reg.index = 0;
+               src.reg.type = -1;
                break;
        }
 
@@ -360,14 +376,35 @@ tgsi_src(struct nvfx_vpc *vpc, const struct tgsi_full_src_register *fsrc) {
        src.swz[1] = fsrc->Register.SwizzleY;
        src.swz[2] = fsrc->Register.SwizzleZ;
        src.swz[3] = fsrc->Register.SwizzleW;
+       src.indirect = 0;
+       src.indirect_reg = 0;
+       src.indirect_swz = 0;
+
+       if(fsrc->Register.Indirect) {
+               if(fsrc->Indirect.File == TGSI_FILE_ADDRESS &&
+                               (fsrc->Register.File == TGSI_FILE_CONSTANT || fsrc->Register.File == TGSI_FILE_INPUT))
+               {
+                       src.indirect = 1;
+                       src.indirect_reg = fsrc->Indirect.Index;
+                       src.indirect_swz = fsrc->Indirect.SwizzleX;
+               }
+               else
+               {
+                       src.reg.index = 0;
+                       src.reg.type = -1;
+               }
+       }
        return src;
 }
 
-static INLINE struct nvfx_sreg
+static INLINE struct nvfx_reg
 tgsi_dst(struct nvfx_vpc *vpc, const struct tgsi_full_dst_register *fdst) {
-       struct nvfx_sreg dst;
+       struct nvfx_reg dst;
 
        switch (fdst->Register.File) {
+       case TGSI_FILE_NULL:
+               dst = nvfx_reg(NVFXSR_NONE, 0);
+               break;
        case TGSI_FILE_OUTPUT:
                dst = vpc->r_result[fdst->Register.Index];
                break;
@@ -378,14 +415,16 @@ tgsi_dst(struct nvfx_vpc *vpc, const struct tgsi_full_dst_register *fdst) {
                dst = vpc->r_address[fdst->Register.Index];
                break;
        default:
-               NOUVEAU_ERR("bad dst file\n");
+               NOUVEAU_ERR("bad dst file %i\n", fdst->Register.File);
+               dst.index = 0;
+               dst.type = 0;
                break;
        }
 
        return dst;
 }
 
-static INLINE int
+static inline int
 tgsi_mask(uint tgsi)
 {
        int mask = 0;
@@ -399,16 +438,20 @@ tgsi_mask(uint tgsi)
 
 static boolean
 nvfx_vertprog_parse_instruction(struct nvfx_context* nvfx, struct nvfx_vpc *vpc,
-                               const struct tgsi_full_instruction *finst)
+                               unsigned idx, const struct tgsi_full_instruction *finst)
 {
-       struct nvfx_sreg src[3], dst, tmp;
-       struct nvfx_sreg none = nvfx_sr(NVFXSR_NONE, 0);
+       struct nvfx_src src[3], tmp;
+       struct nvfx_reg dst;
+       struct nvfx_reg final_dst;
+       struct nvfx_src none = nvfx_src(nvfx_reg(NVFXSR_NONE, 0));
+       struct nvfx_insn insn;
+       struct nvfx_relocation reloc;
+       struct nvfx_loop_entry loop;
+       boolean sat = FALSE;
        int mask;
        int ai = -1, ci = -1, ii = -1;
        int i;
-
-       if (finst->Instruction.Opcode == TGSI_OPCODE_END)
-               return TRUE;
+       unsigned sub_depth = 0;
 
        for (i = 0; i < finst->Instruction.NumSrcRegs; i++) {
                const struct tgsi_full_src_register *fsrc;
@@ -430,9 +473,8 @@ nvfx_vertprog_parse_instruction(struct nvfx_context* nvfx, struct nvfx_vpc *vpc,
                                ai = fsrc->Register.Index;
                                src[i] = tgsi_src(vpc, fsrc);
                        } else {
-                               src[i] = temp(vpc);
-                               arith(vpc, VEC, MOV, src[i], NVFX_VP_MASK_ALL,
-                                     tgsi_src(vpc, fsrc), none, none);
+                               src[i] = nvfx_src(temp(vpc));
+                               nvfx_vp_emit(vpc, arith(0, VEC, MOV, src[i].reg, NVFX_VP_MASK_ALL, tgsi_src(vpc, fsrc), none, none));
                        }
                        break;
                case TGSI_FILE_CONSTANT:
@@ -441,9 +483,8 @@ nvfx_vertprog_parse_instruction(struct nvfx_context* nvfx, struct nvfx_vpc *vpc,
                                ci = fsrc->Register.Index;
                                src[i] = tgsi_src(vpc, fsrc);
                        } else {
-                               src[i] = temp(vpc);
-                               arith(vpc, VEC, MOV, src[i], NVFX_VP_MASK_ALL,
-                                     tgsi_src(vpc, fsrc), none, none);
+                               src[i] = nvfx_src(temp(vpc));
+                               nvfx_vp_emit(vpc, arith(0, VEC, MOV, src[i].reg, NVFX_VP_MASK_ALL, tgsi_src(vpc, fsrc), none, none));
                        }
                        break;
                case TGSI_FILE_IMMEDIATE:
@@ -452,9 +493,8 @@ nvfx_vertprog_parse_instruction(struct nvfx_context* nvfx, struct nvfx_vpc *vpc,
                                ii = fsrc->Register.Index;
                                src[i] = tgsi_src(vpc, fsrc);
                        } else {
-                               src[i] = temp(vpc);
-                               arith(vpc, VEC, MOV, src[i], NVFX_VP_MASK_ALL,
-                                     tgsi_src(vpc, fsrc), none, none);
+                               src[i] = nvfx_src(temp(vpc));
+                               nvfx_vp_emit(vpc, arith(0, VEC, MOV, src[i].reg, NVFX_VP_MASK_ALL, tgsi_src(vpc, fsrc), none, none));
                        }
                        break;
                case TGSI_FILE_TEMPORARY:
@@ -466,109 +506,294 @@ nvfx_vertprog_parse_instruction(struct nvfx_context* nvfx, struct nvfx_vpc *vpc,
                }
        }
 
-       dst  = tgsi_dst(vpc, &finst->Dst[0]);
+       for (i = 0; i < finst->Instruction.NumSrcRegs; i++) {
+               if(src[i].reg.type < 0)
+                       return FALSE;
+       }
+
+       if(finst->Dst[0].Register.File == TGSI_FILE_ADDRESS &&
+                       finst->Instruction.Opcode != TGSI_OPCODE_ARL)
+               return FALSE;
+
+       final_dst = dst  = tgsi_dst(vpc, &finst->Dst[0]);
        mask = tgsi_mask(finst->Dst[0].Register.WriteMask);
+       if(finst->Instruction.Saturate == TGSI_SAT_ZERO_ONE)
+       {
+               assert(finst->Instruction.Opcode != TGSI_OPCODE_ARL);
+               if(nvfx->use_nv4x)
+                       sat = TRUE;
+               else if(dst.type != NVFXSR_TEMP)
+                       dst = temp(vpc);
+       }
 
        switch (finst->Instruction.Opcode) {
        case TGSI_OPCODE_ABS:
-               arith(vpc, VEC, MOV, dst, mask, abs(src[0]), none, none);
+               nvfx_vp_emit(vpc, arith(sat, VEC, MOV, dst, mask, abs(src[0]), none, none));
                break;
        case TGSI_OPCODE_ADD:
-               arith(vpc, VEC, ADD, dst, mask, src[0], none, src[1]);
+               nvfx_vp_emit(vpc, arith(sat, VEC, ADD, dst, mask, src[0], none, src[1]));
                break;
        case TGSI_OPCODE_ARL:
-               arith(vpc, VEC, ARL, dst, mask, src[0], none, none);
+               nvfx_vp_emit(vpc, arith(0, VEC, ARL, dst, mask, src[0], none, none));
                break;
+       case TGSI_OPCODE_CMP:
+               insn = arith(0, VEC, MOV, none.reg, mask, src[0], none, none);
+               insn.cc_update = 1;
+               nvfx_vp_emit(vpc, insn);
+
+               insn = arith(sat, VEC, MOV, dst, mask, src[2], none, none);
+               insn.cc_test = NVFX_COND_GE;
+               nvfx_vp_emit(vpc, insn);
+
+               insn = arith(sat, VEC, MOV, dst, mask, src[1], none, none);
+               insn.cc_test = NVFX_COND_LT;
+               nvfx_vp_emit(vpc, insn);
+               break;
+       case TGSI_OPCODE_COS:
+               nvfx_vp_emit(vpc, arith(sat, SCA, COS, dst, mask, none, none, src[0]));
+               break;
+        case TGSI_OPCODE_DP2:
+                tmp = nvfx_src(temp(vpc));
+                nvfx_vp_emit(vpc, arith(0, VEC, MUL, tmp.reg, NVFX_VP_MASK_X | NVFX_VP_MASK_Y, src[0], src[1], none));
+                nvfx_vp_emit(vpc, arith(sat, VEC, ADD, dst, mask, swz(tmp, X, X, X, X), none, swz(tmp, Y, Y, Y, Y)));
+                break;
        case TGSI_OPCODE_DP3:
-               arith(vpc, VEC, DP3, dst, mask, src[0], src[1], none);
+               nvfx_vp_emit(vpc, arith(sat, VEC, DP3, dst, mask, src[0], src[1], none));
                break;
        case TGSI_OPCODE_DP4:
-               arith(vpc, VEC, DP4, dst, mask, src[0], src[1], none);
+               nvfx_vp_emit(vpc, arith(sat, VEC, DP4, dst, mask, src[0], src[1], none));
                break;
        case TGSI_OPCODE_DPH:
-               arith(vpc, VEC, DPH, dst, mask, src[0], src[1], none);
+               nvfx_vp_emit(vpc, arith(sat, VEC, DPH, dst, mask, src[0], src[1], none));
                break;
        case TGSI_OPCODE_DST:
-               arith(vpc, VEC, DST, dst, mask, src[0], src[1], none);
+               nvfx_vp_emit(vpc, arith(sat, VEC, DST, dst, mask, src[0], src[1], none));
                break;
        case TGSI_OPCODE_EX2:
-               arith(vpc, SCA, EX2, dst, mask, none, none, src[0]);
+               nvfx_vp_emit(vpc, arith(sat, SCA, EX2, dst, mask, none, none, src[0]));
                break;
        case TGSI_OPCODE_EXP:
-               arith(vpc, SCA, EXP, dst, mask, none, none, src[0]);
+               nvfx_vp_emit(vpc, arith(sat, SCA, EXP, dst, mask, none, none, src[0]));
                break;
        case TGSI_OPCODE_FLR:
-               arith(vpc, VEC, FLR, dst, mask, src[0], none, none);
+               nvfx_vp_emit(vpc, arith(sat, VEC, FLR, dst, mask, src[0], none, none));
                break;
        case TGSI_OPCODE_FRC:
-               arith(vpc, VEC, FRC, dst, mask, src[0], none, none);
+               nvfx_vp_emit(vpc, arith(sat, VEC, FRC, dst, mask, src[0], none, none));
                break;
        case TGSI_OPCODE_LG2:
-               arith(vpc, SCA, LG2, dst, mask, none, none, src[0]);
+               nvfx_vp_emit(vpc, arith(sat, SCA, LG2, dst, mask, none, none, src[0]));
                break;
        case TGSI_OPCODE_LIT:
-               arith(vpc, SCA, LIT, dst, mask, none, none, src[0]);
+               nvfx_vp_emit(vpc, arith(sat, SCA, LIT, dst, mask, none, none, src[0]));
                break;
        case TGSI_OPCODE_LOG:
-               arith(vpc, SCA, LOG, dst, mask, none, none, src[0]);
+               nvfx_vp_emit(vpc, arith(sat, SCA, LOG, dst, mask, none, none, src[0]));
+               break;
+       case TGSI_OPCODE_LRP:
+               tmp = nvfx_src(temp(vpc));
+               nvfx_vp_emit(vpc, arith(0, VEC, MAD, tmp.reg, mask, neg(src[0]), src[2], src[2]));
+               nvfx_vp_emit(vpc, arith(sat, VEC, MAD, dst, mask, src[0], src[1], tmp));
                break;
        case TGSI_OPCODE_MAD:
-               arith(vpc, VEC, MAD, dst, mask, src[0], src[1], src[2]);
+               nvfx_vp_emit(vpc, arith(sat, VEC, MAD, dst, mask, src[0], src[1], src[2]));
                break;
        case TGSI_OPCODE_MAX:
-               arith(vpc, VEC, MAX, dst, mask, src[0], src[1], none);
+               nvfx_vp_emit(vpc, arith(sat, VEC, MAX, dst, mask, src[0], src[1], none));
                break;
        case TGSI_OPCODE_MIN:
-               arith(vpc, VEC, MIN, dst, mask, src[0], src[1], none);
+               nvfx_vp_emit(vpc, arith(sat, VEC, MIN, dst, mask, src[0], src[1], none));
                break;
        case TGSI_OPCODE_MOV:
-               arith(vpc, VEC, MOV, dst, mask, src[0], none, none);
+               nvfx_vp_emit(vpc, arith(sat, VEC, MOV, dst, mask, src[0], none, none));
                break;
        case TGSI_OPCODE_MUL:
-               arith(vpc, VEC, MUL, dst, mask, src[0], src[1], none);
+               nvfx_vp_emit(vpc, arith(sat, VEC, MUL, dst, mask, src[0], src[1], none));
+               break;
+       case TGSI_OPCODE_NOP:
                break;
        case TGSI_OPCODE_POW:
-               tmp = temp(vpc);
-               arith(vpc, SCA, LG2, tmp, NVFX_VP_MASK_X, none, none,
-                     swz(src[0], X, X, X, X));
-               arith(vpc, VEC, MUL, tmp, NVFX_VP_MASK_X, swz(tmp, X, X, X, X),
-                     swz(src[1], X, X, X, X), none);
-               arith(vpc, SCA, EX2, dst, mask, none, none,
-                     swz(tmp, X, X, X, X));
+               tmp = nvfx_src(temp(vpc));
+               nvfx_vp_emit(vpc, arith(0, SCA, LG2, tmp.reg, NVFX_VP_MASK_X, none, none, swz(src[0], X, X, X, X)));
+               nvfx_vp_emit(vpc, arith(0, VEC, MUL, tmp.reg, NVFX_VP_MASK_X, swz(tmp, X, X, X, X), swz(src[1], X, X, X, X), none));
+               nvfx_vp_emit(vpc, arith(sat, SCA, EX2, dst, mask, none, none, swz(tmp, X, X, X, X)));
                break;
        case TGSI_OPCODE_RCP:
-               arith(vpc, SCA, RCP, dst, mask, none, none, src[0]);
-               break;
-       case TGSI_OPCODE_RET:
+               nvfx_vp_emit(vpc, arith(sat, SCA, RCP, dst, mask, none, none, src[0]));
                break;
        case TGSI_OPCODE_RSQ:
-               arith(vpc, SCA, RSQ, dst, mask, none, none, abs(src[0]));
+               nvfx_vp_emit(vpc, arith(sat, SCA, RSQ, dst, mask, none, none, abs(src[0])));
+               break;
+       case TGSI_OPCODE_SEQ:
+               nvfx_vp_emit(vpc, arith(sat, VEC, SEQ, dst, mask, src[0], src[1], none));
+               break;
+       case TGSI_OPCODE_SFL:
+               nvfx_vp_emit(vpc, arith(sat, VEC, SFL, dst, mask, src[0], src[1], none));
                break;
        case TGSI_OPCODE_SGE:
-               arith(vpc, VEC, SGE, dst, mask, src[0], src[1], none);
+               nvfx_vp_emit(vpc, arith(sat, VEC, SGE, dst, mask, src[0], src[1], none));
                break;
        case TGSI_OPCODE_SGT:
-               arith(vpc, VEC, SGT, dst, mask, src[0], src[1], none);
+               nvfx_vp_emit(vpc, arith(sat, VEC, SGT, dst, mask, src[0], src[1], none));
+               break;
+       case TGSI_OPCODE_SIN:
+               nvfx_vp_emit(vpc, arith(sat, SCA, SIN, dst, mask, none, none, src[0]));
+               break;
+       case TGSI_OPCODE_SLE:
+               nvfx_vp_emit(vpc, arith(sat, VEC, SLE, dst, mask, src[0], src[1], none));
                break;
        case TGSI_OPCODE_SLT:
-               arith(vpc, VEC, SLT, dst, mask, src[0], src[1], none);
+               nvfx_vp_emit(vpc, arith(sat, VEC, SLT, dst, mask, src[0], src[1], none));
+               break;
+       case TGSI_OPCODE_SNE:
+               nvfx_vp_emit(vpc, arith(sat, VEC, SNE, dst, mask, src[0], src[1], none));
+               break;
+       case TGSI_OPCODE_SSG:
+               nvfx_vp_emit(vpc, arith(sat, VEC, SSG, dst, mask, src[0], src[1], none));
+               break;
+       case TGSI_OPCODE_STR:
+               nvfx_vp_emit(vpc, arith(sat, VEC, STR, dst, mask, src[0], src[1], none));
                break;
        case TGSI_OPCODE_SUB:
-               arith(vpc, VEC, ADD, dst, mask, src[0], none, neg(src[1]));
+               nvfx_vp_emit(vpc, arith(sat, VEC, ADD, dst, mask, src[0], none, neg(src[1])));
                break;
+        case TGSI_OPCODE_TRUNC:
+                tmp = nvfx_src(temp(vpc));
+                insn = arith(0, VEC, MOV, none.reg, mask, src[0], none, none);
+                insn.cc_update = 1;
+                nvfx_vp_emit(vpc, insn);
+
+                nvfx_vp_emit(vpc, arith(0, VEC, FLR, tmp.reg, mask, abs(src[0]), none, none));
+                nvfx_vp_emit(vpc, arith(sat, VEC, MOV, dst, mask, tmp, none, none));
+
+                insn = arith(sat, VEC, MOV, dst, mask, neg(tmp), none, none);
+                insn.cc_test = NVFX_COND_LT;
+                nvfx_vp_emit(vpc, insn);
+                break;
        case TGSI_OPCODE_XPD:
-               tmp = temp(vpc);
-               arith(vpc, VEC, MUL, tmp, mask,
-                     swz(src[0], Z, X, Y, Y), swz(src[1], Y, Z, X, X), none);
-               arith(vpc, VEC, MAD, dst, (mask & ~NVFX_VP_MASK_W),
-                     swz(src[0], Y, Z, X, X), swz(src[1], Z, X, Y, Y),
-                     neg(tmp));
+               tmp = nvfx_src(temp(vpc));
+               nvfx_vp_emit(vpc, arith(0, VEC, MUL, tmp.reg, mask, swz(src[0], Z, X, Y, Y), swz(src[1], Y, Z, X, X), none));
+               nvfx_vp_emit(vpc, arith(sat, VEC, MAD, dst, (mask & ~NVFX_VP_MASK_W), swz(src[0], Y, Z, X, X), swz(src[1], Z, X, Y, Y), neg(tmp)));
+               break;
+
+       case TGSI_OPCODE_IF:
+               insn = arith(0, VEC, MOV, none.reg, NVFX_VP_MASK_X, src[0], none, none);
+               insn.cc_update = 1;
+               nvfx_vp_emit(vpc, insn);
+
+               reloc.location = vpc->vp->nr_insns;
+               reloc.target = finst->Label.Label + 1;
+               util_dynarray_append(&vpc->label_relocs, struct nvfx_relocation, reloc);
+
+               insn = arith(0, SCA, BRA, none.reg, 0, none, none, none);
+               insn.cc_test = NVFX_COND_EQ;
+               insn.cc_swz[0] = insn.cc_swz[1] = insn.cc_swz[2] = insn.cc_swz[3] = 0;
+               nvfx_vp_emit(vpc, insn);
+               break;
+
+       case TGSI_OPCODE_ELSE:
+       case TGSI_OPCODE_BRA:
+       case TGSI_OPCODE_CAL:
+               reloc.location = vpc->vp->nr_insns;
+               reloc.target = finst->Label.Label;
+               util_dynarray_append(&vpc->label_relocs, struct nvfx_relocation, reloc);
+
+               if(finst->Instruction.Opcode == TGSI_OPCODE_CAL)
+                       insn = arith(0, SCA, CAL, none.reg, 0, none, none, none);
+               else
+                       insn = arith(0, SCA, BRA, none.reg, 0, none, none, none);
+               nvfx_vp_emit(vpc, insn);
+               break;
+
+       case TGSI_OPCODE_RET:
+               if(sub_depth || !nvfx->use_vp_clipping) {
+                       tmp = none;
+                       tmp.swz[0] = tmp.swz[1] = tmp.swz[2] = tmp.swz[3] = 0;
+                       nvfx_vp_emit(vpc, arith(0, SCA, RET, none.reg, 0, none, none, tmp));
+               } else {
+                       reloc.location = vpc->vp->nr_insns;
+                       reloc.target = vpc->info->num_instructions;
+                       util_dynarray_append(&vpc->label_relocs, struct nvfx_relocation, reloc);
+                       nvfx_vp_emit(vpc, arith(0, SCA, BRA, none.reg, 0, none, none, none));
+               }
+               break;
+
+       case TGSI_OPCODE_BGNSUB:
+               ++sub_depth;
+               break;
+       case TGSI_OPCODE_ENDSUB:
+               --sub_depth;
+               break;
+       case TGSI_OPCODE_ENDIF:
+               /* nothing to do here */
+               break;
+
+       case TGSI_OPCODE_BGNLOOP:
+               loop.cont_target = idx;
+               loop.brk_target = finst->Label.Label + 1;
+               util_dynarray_append(&vpc->loop_stack, struct nvfx_loop_entry, loop);
+               break;
+
+       case TGSI_OPCODE_ENDLOOP:
+               loop = util_dynarray_pop(&vpc->loop_stack, struct nvfx_loop_entry);
+
+               reloc.location = vpc->vp->nr_insns;
+               reloc.target = loop.cont_target;
+               util_dynarray_append(&vpc->label_relocs, struct nvfx_relocation, reloc);
+
+               nvfx_vp_emit(vpc, arith(0, SCA, BRA, none.reg, 0, none, none, none));
+               break;
+
+       case TGSI_OPCODE_CONT:
+               loop = util_dynarray_top(&vpc->loop_stack, struct nvfx_loop_entry);
+
+               reloc.location = vpc->vp->nr_insns;
+               reloc.target = loop.cont_target;
+               util_dynarray_append(&vpc->label_relocs, struct nvfx_relocation, reloc);
+
+               nvfx_vp_emit(vpc, arith(0, SCA, BRA, none.reg, 0, none, none, none));
+               break;
+
+       case TGSI_OPCODE_BRK:
+               loop = util_dynarray_top(&vpc->loop_stack, struct nvfx_loop_entry);
+
+               reloc.location = vpc->vp->nr_insns;
+               reloc.target = loop.brk_target;
+               util_dynarray_append(&vpc->label_relocs, struct nvfx_relocation, reloc);
+
+               nvfx_vp_emit(vpc, arith(0, SCA, BRA, none.reg, 0, none, none, none));
+               break;
+
+       case TGSI_OPCODE_END:
+               assert(!sub_depth);
+               if(nvfx->use_vp_clipping) {
+                       if(idx != (vpc->info->num_instructions - 1)) {
+                               reloc.location = vpc->vp->nr_insns;
+                               reloc.target = vpc->info->num_instructions;
+                               util_dynarray_append(&vpc->label_relocs, struct nvfx_relocation, reloc);
+                               nvfx_vp_emit(vpc, arith(0, SCA, BRA, none.reg, 0, none, none, none));
+                       }
+               } else {
+                       if(vpc->vp->nr_insns)
+                               vpc->vp->insns[vpc->vp->nr_insns - 1].data[3] |= NVFX_VP_INST_LAST;
+                       nvfx_vp_emit(vpc, arith(0, VEC, NOP, none.reg, 0, none, none, none));
+                       vpc->vp->insns[vpc->vp->nr_insns - 1].data[3] |= NVFX_VP_INST_LAST;
+               }
                break;
+
        default:
                NOUVEAU_ERR("invalid opcode %d\n", finst->Instruction.Opcode);
                return FALSE;
        }
 
+       if(finst->Instruction.Saturate == TGSI_SAT_ZERO_ONE && !nvfx->use_nv4x)
+       {
+               if(!vpc->r_0_1.type)
+                       vpc->r_0_1 = constant(vpc, -1, 0, 1, 0, 0);
+               nvfx_vp_emit(vpc, arith(0, VEC, MAX, dst, mask, nvfx_src(dst), swz(nvfx_src(vpc->r_0_1), X, X, X, X), none));
+               nvfx_vp_emit(vpc, arith(0, VEC, MIN, final_dst, mask, nvfx_src(dst), swz(nvfx_src(vpc->r_0_1), Y, Y, Y, Y), none));
+       }
+
        release_temps(vpc);
        return TRUE;
 }
@@ -614,12 +839,13 @@ nvfx_vertprog_parse_decl_output(struct nvfx_context* nvfx, struct nvfx_vpc *vpc,
                hw = NVFX_VP(INST_DEST_PSZ);
                break;
        case TGSI_SEMANTIC_GENERIC:
-               if (fdec->Semantic.Index <= 7) {
-                       hw = NVFX_VP(INST_DEST_TC(fdec->Semantic.Index));
-               } else {
-                       NOUVEAU_ERR("bad generic semantic index\n");
-                       return FALSE;
-               }
+               hw = (vpc->vp->generic_to_fp_input[fdec->Semantic.Index] & 0xf) - NVFX_FP_OP_INPUT_SRC_TC(0);
+               if(hw <= 8)
+                       hw = NVFX_VP(INST_DEST_TC(hw));
+               else if(hw == 9) /* TODO: this is correct, but how does this overlapping work exactly? */
+                       hw = NV40_VP_INST_DEST_PSZ;
+               else
+                       assert(0);
                break;
        case TGSI_SEMANTIC_EDGEFLAG:
                /* not really an error just a fallback */
@@ -630,7 +856,7 @@ nvfx_vertprog_parse_decl_output(struct nvfx_context* nvfx, struct nvfx_vpc *vpc,
                return FALSE;
        }
 
-       vpc->r_result[idx] = nvfx_sr(NVFXSR_OUTPUT, hw);
+       vpc->r_result[idx] = nvfx_reg(NVFXSR_OUTPUT, hw);
        return TRUE;
 }
 
@@ -638,9 +864,40 @@ static boolean
 nvfx_vertprog_prepare(struct nvfx_context* nvfx, struct nvfx_vpc *vpc)
 {
        struct tgsi_parse_context p;
-       int high_temp = -1, high_addr = -1, nr_imm = 0, i;
+       int high_const = -1, high_temp = -1, high_addr = -1, nr_imm = 0, i;
+       struct util_semantic_set set;
+       unsigned char sem_layout[10];
+       unsigned num_outputs;
+       unsigned num_texcoords = nvfx->is_nv4x ? 10 : 8;
+
+       num_outputs = util_semantic_set_from_program_file(&set, vpc->pipe.tokens, TGSI_FILE_OUTPUT);
 
-       tgsi_parse_init(&p, vpc->vp->pipe.tokens);
+       if(num_outputs > num_texcoords) {
+               NOUVEAU_ERR("too many vertex program outputs: %i\n", num_outputs);
+               return FALSE;
+       }
+       util_semantic_layout_from_set(sem_layout, &set, num_texcoords, num_texcoords);
+
+       /* hope 0xf is (0, 0, 0, 1) initialized; otherwise, we are _probably_ not required to do this */
+       memset(vpc->vp->generic_to_fp_input, 0x0f, sizeof(vpc->vp->generic_to_fp_input));
+       for(int i = 0; i < num_texcoords; ++i) {
+               if(sem_layout[i] == 0xff)
+                       continue;
+               //printf("vp: GENERIC[%i] to fpreg %i\n", sem_layout[i], NVFX_FP_OP_INPUT_SRC_TC(0) + i);
+               vpc->vp->generic_to_fp_input[sem_layout[i]] = 0xf0 | NVFX_FP_OP_INPUT_SRC_TC(i);
+       }
+
+       vpc->vp->sprite_fp_input = -1;
+       for(int i = 0; i < num_texcoords; ++i)
+       {
+               if(sem_layout[i] == 0xff)
+               {
+                       vpc->vp->sprite_fp_input = NVFX_FP_OP_INPUT_SRC_TC(i);
+                       break;
+               }
+       }
+
+       tgsi_parse_init(&p, vpc->pipe.tokens);
        while (!tgsi_parse_end_of_tokens(&p)) {
                const union tgsi_full_token *tok = &p.FullToken;
 
@@ -661,14 +918,18 @@ nvfx_vertprog_prepare(struct nvfx_context* nvfx, struct nvfx_vpc *vpc)
                                                fdec->Range.Last;
                                }
                                break;
-#if 0 /* this would be nice.. except gallium doesn't track it */
                        case TGSI_FILE_ADDRESS:
                                if (fdec->Range.Last > high_addr) {
                                        high_addr =
                                                fdec->Range.Last;
                                }
                                break;
-#endif
+                       case TGSI_FILE_CONSTANT:
+                               if (fdec->Range.Last > high_const) {
+                                       high_const =
+                                                       fdec->Range.Last;
+                               }
+                               break;
                        case TGSI_FILE_OUTPUT:
                                if (!nvfx_vertprog_parse_decl_output(nvfx, vpc, fdec))
                                        return FALSE;
@@ -678,23 +939,6 @@ nvfx_vertprog_prepare(struct nvfx_context* nvfx, struct nvfx_vpc *vpc)
                        }
                }
                        break;
-#if 1 /* yay, parse instructions looking for address regs instead */
-               case TGSI_TOKEN_TYPE_INSTRUCTION:
-               {
-                       const struct tgsi_full_instruction *finst;
-                       const struct tgsi_full_dst_register *fdst;
-
-                       finst = &p.FullToken.FullInstruction;
-                       fdst = &finst->Dst[0];
-
-                       if (fdst->Register.File == TGSI_FILE_ADDRESS) {
-                               if (fdst->Register.Index > high_addr)
-                                       high_addr = fdst->Register.Index;
-                       }
-
-               }
-                       break;
-#endif
                default:
                        break;
                }
@@ -702,56 +946,88 @@ nvfx_vertprog_prepare(struct nvfx_context* nvfx, struct nvfx_vpc *vpc)
        tgsi_parse_free(&p);
 
        if (nr_imm) {
-               vpc->imm = CALLOC(nr_imm, sizeof(struct nvfx_sreg));
+               vpc->imm = CALLOC(nr_imm, sizeof(struct nvfx_reg));
                assert(vpc->imm);
        }
 
        if (++high_temp) {
-               vpc->r_temp = CALLOC(high_temp, sizeof(struct nvfx_sreg));
+               vpc->r_temp = CALLOC(high_temp, sizeof(struct nvfx_reg));
                for (i = 0; i < high_temp; i++)
                        vpc->r_temp[i] = temp(vpc);
        }
 
        if (++high_addr) {
-               vpc->r_address = CALLOC(high_addr, sizeof(struct nvfx_sreg));
+               vpc->r_address = CALLOC(high_addr, sizeof(struct nvfx_reg));
                for (i = 0; i < high_addr; i++)
-                       vpc->r_address[i] = temp(vpc);
+                       vpc->r_address[i] = nvfx_reg(NVFXSR_TEMP, i);
+       }
+
+       if(++high_const) {
+               vpc->r_const = CALLOC(high_const, sizeof(struct nvfx_reg));
+               for (i = 0; i < high_const; i++)
+                       vpc->r_const[i] = constant(vpc, i, 0, 0, 0, 0);
        }
 
        vpc->r_temps_discard = 0;
        return TRUE;
 }
 
-static void
-nvfx_vertprog_translate(struct nvfx_context *nvfx,
-                       struct nvfx_vertex_program *vp)
+DEBUG_GET_ONCE_BOOL_OPTION(nvfx_dump_vp, "NVFX_DUMP_VP", FALSE)
+
+static struct nvfx_vertex_program*
+nvfx_vertprog_translate(struct nvfx_context *nvfx, const struct pipe_shader_state* vps, struct tgsi_shader_info* info)
 {
        struct tgsi_parse_context parse;
+       struct nvfx_vertex_program* vp = NULL;
        struct nvfx_vpc *vpc = NULL;
-       struct nvfx_sreg none = nvfx_sr(NVFXSR_NONE, 0);
+       struct nvfx_src none = nvfx_src(nvfx_reg(NVFXSR_NONE, 0));
+       struct util_dynarray insns;
        int i;
 
-       vpc = CALLOC(1, sizeof(struct nvfx_vpc));
+       tgsi_parse_init(&parse, vps->tokens);
+
+       vp = CALLOC_STRUCT(nvfx_vertex_program);
+       if(!vp)
+               goto out_err;
+
+       vpc = CALLOC_STRUCT(nvfx_vpc);
        if (!vpc)
-               return;
+               goto out_err;
+
+       vpc->nvfx = nvfx;
        vpc->vp = vp;
+       vpc->pipe = *vps;
+       vpc->info = info;
+
+       {
+               // TODO: use a 64-bit atomic here!
+               static unsigned long long id = 0;
+               vp->id = ++id;
+       }
+
+       /* reserve space for ucps */
+       if(nvfx->use_vp_clipping)
+       {
+               for(i = 0; i < 6; ++i)
+                       constant(vpc, -1, 0, 0, 0, 0);
+       }
 
        if (!nvfx_vertprog_prepare(nvfx, vpc)) {
                FREE(vpc);
-               return;
+               return NULL;
        }
 
        /* Redirect post-transform vertex position to a temp if user clip
         * planes are enabled.  We need to append code to the vtxprog
         * to handle clip planes later.
         */
-       if (vp->ucp.nr)  {
+       /* TODO: maybe support patching this depending on whether there are ucps: not sure if it is really matters much */
+       if (nvfx->use_vp_clipping)  {
                vpc->r_result[vpc->hpos_idx] = temp(vpc);
                vpc->r_temps_discard = 0;
        }
 
-       tgsi_parse_init(&parse, vp->pipe.tokens);
-
+       util_dynarray_init(&insns);
        while (!tgsi_parse_end_of_tokens(&parse)) {
                tgsi_parse_token(&parse);
 
@@ -774,8 +1050,10 @@ nvfx_vertprog_translate(struct nvfx_context *nvfx,
                case TGSI_TOKEN_TYPE_INSTRUCTION:
                {
                        const struct tgsi_full_instruction *finst;
+                       unsigned idx = insns.size >> 2;
+                       util_dynarray_append(&insns, unsigned, vp->nr_insns);
                        finst = &parse.FullToken.FullInstruction;
-                       if (!nvfx_vertprog_parse_instruction(nvfx, vpc, finst))
+                       if (!nvfx_vertprog_parse_instruction(nvfx, vpc, idx, finst))
                                goto out_err;
                }
                        break;
@@ -784,95 +1062,168 @@ nvfx_vertprog_translate(struct nvfx_context *nvfx,
                }
        }
 
+       util_dynarray_append(&insns, unsigned, vp->nr_insns);
+
+       for(unsigned i = 0; i < vpc->label_relocs.size; i += sizeof(struct nvfx_relocation))
+       {
+               struct nvfx_relocation* label_reloc = (struct nvfx_relocation*)((char*)vpc->label_relocs.data + i);
+               struct nvfx_relocation hw_reloc;
+
+               hw_reloc.location = label_reloc->location;
+               hw_reloc.target = ((unsigned*)insns.data)[label_reloc->target];
+
+               //debug_printf("hw %u -> tgsi %u = hw %u\n", hw_reloc.location, label_reloc->target, hw_reloc.target);
+
+               util_dynarray_append(&vp->branch_relocs, struct nvfx_relocation, hw_reloc);
+       }
+       util_dynarray_fini(&insns);
+       util_dynarray_trim(&vp->branch_relocs);
+
+       /* XXX: what if we add a RET before?!  make sure we jump here...*/
+
        /* Write out HPOS if it was redirected to a temp earlier */
        if (vpc->r_result[vpc->hpos_idx].type != NVFXSR_OUTPUT) {
-               struct nvfx_sreg hpos = nvfx_sr(NVFXSR_OUTPUT,
+               struct nvfx_reg hpos = nvfx_reg(NVFXSR_OUTPUT,
                                                NVFX_VP(INST_DEST_POS));
-               struct nvfx_sreg htmp = vpc->r_result[vpc->hpos_idx];
+               struct nvfx_src htmp = nvfx_src(vpc->r_result[vpc->hpos_idx]);
 
-               arith(vpc, VEC, MOV, hpos, NVFX_VP_MASK_ALL, htmp, none, none);
+               nvfx_vp_emit(vpc, arith(0, VEC, MOV, hpos, NVFX_VP_MASK_ALL, htmp, none, none));
        }
 
        /* Insert code to handle user clip planes */
-       for (i = 0; i < vp->ucp.nr; i++) {
-               struct nvfx_sreg cdst = nvfx_sr(NVFXSR_OUTPUT,
-                                               NVFX_VP_INST_DEST_CLIP(i));
-               struct nvfx_sreg ceqn = constant(vpc, -1,
-                                                nvfx->clip.ucp[i][0],
-                                                nvfx->clip.ucp[i][1],
-                                                nvfx->clip.ucp[i][2],
-                                                nvfx->clip.ucp[i][3]);
-               struct nvfx_sreg htmp = vpc->r_result[vpc->hpos_idx];
-               unsigned mask;
-
-               switch (i) {
-               case 0: case 3: mask = NVFX_VP_MASK_Y; break;
-               case 1: case 4: mask = NVFX_VP_MASK_Z; break;
-               case 2: case 5: mask = NVFX_VP_MASK_W; break;
-               default:
-                       NOUVEAU_ERR("invalid clip dist #%d\n", i);
-                       goto out_err;
+       if(nvfx->use_vp_clipping)
+       {
+               for (i = 0; i < 6; i++) {
+                       struct nvfx_reg cdst = nvfx_reg(NVFXSR_OUTPUT, NV30_VP_INST_DEST_CLP(i));
+                       struct nvfx_src ceqn = nvfx_src(nvfx_reg(NVFXSR_CONST, i));
+                       struct nvfx_src htmp = nvfx_src(vpc->r_result[vpc->hpos_idx]);
+                       unsigned mask;
+
+                       if(nvfx->is_nv4x)
+                       {
+                               switch (i) {
+                               case 0: case 3: mask = NVFX_VP_MASK_Y; break;
+                               case 1: case 4: mask = NVFX_VP_MASK_Z; break;
+                               case 2: case 5: mask = NVFX_VP_MASK_W; break;
+                               default:
+                                       NOUVEAU_ERR("invalid clip dist #%d\n", i);
+                                       goto out_err;
+                               }
+                       }
+                       else
+                               mask = NVFX_VP_MASK_X;
+
+                       nvfx_vp_emit(vpc, arith(0, VEC, DP4, cdst, mask, htmp, ceqn, none));
                }
+       }
+
+       if(debug_get_option_nvfx_dump_vp())
+       {
+               debug_printf("\n");
+               tgsi_dump(vpc->pipe.tokens, 0);
 
-               arith(vpc, VEC, DP4, cdst, mask, htmp, ceqn, none);
+               debug_printf("\n%s vertex program:\n", nvfx->is_nv4x ? "nv4x" : "nv3x");
+               for (i = 0; i < vp->nr_insns; i++)
+                       debug_printf("%3u: %08x %08x %08x %08x\n", i, vp->insns[i].data[0], vp->insns[i].data[1], vp->insns[i].data[2], vp->insns[i].data[3]);
+               debug_printf("\n");
        }
 
-       vp->insns[vp->nr_insns - 1].data[3] |= NVFX_VP_INST_LAST;
-       vp->translated = TRUE;
-out_err:
+       vp->clip_nr = -1;
+       vp->exec_start = -1;
+
+out:
        tgsi_parse_free(&parse);
-       if (vpc->r_temp)
+       if(vpc) {
+               util_dynarray_fini(&vpc->label_relocs);
+               util_dynarray_fini(&vpc->loop_stack);
                FREE(vpc->r_temp);
-       if (vpc->r_address)
                FREE(vpc->r_address);
-       if (vpc->imm)
+               FREE(vpc->r_const);
                FREE(vpc->imm);
-       FREE(vpc);
+               FREE(vpc);
+       }
+       return vp;
+
+out_err:
+       FREE(vp);
+       vp = NULL;
+       goto out;
 }
 
-static boolean
+static struct nvfx_vertex_program*
+nvfx_vertprog_translate_draw_vp(struct nvfx_context *nvfx, struct nvfx_pipe_vertex_program* pvp)
+{
+       struct nvfx_vertex_program* vp = NULL;
+       struct pipe_shader_state vps;
+       struct tgsi_shader_info info;
+       struct ureg_program *ureg = NULL;
+       unsigned num_outputs = MIN2(pvp->info.num_outputs, 16);
+
+       ureg = ureg_create( TGSI_PROCESSOR_VERTEX );
+       if(ureg == NULL)
+               return 0;
+
+       for (unsigned i = 0; i < num_outputs; i++)
+               ureg_MOV(ureg, ureg_DECL_output(ureg, pvp->info.output_semantic_name[i], pvp->info.output_semantic_index[i]), ureg_DECL_vs_input(ureg, i));
+
+       ureg_END( ureg );
+
+       vps.tokens = ureg_get_tokens(ureg, 0);
+       tgsi_scan_shader(vps.tokens, &info);
+       vp = nvfx_vertprog_translate(nvfx, &vps, &info);
+       ureg_free_tokens(vps.tokens);
+       ureg_destroy(ureg);
+
+       return vp;
+}
+
+boolean
 nvfx_vertprog_validate(struct nvfx_context *nvfx)
 {
-       struct pipe_context *pipe = &nvfx->pipe;
        struct nvfx_screen *screen = nvfx->screen;
        struct nouveau_channel *chan = screen->base.channel;
        struct nouveau_grobj *eng3d = screen->eng3d;
-       struct nvfx_vertex_program *vp;
+       struct nvfx_pipe_vertex_program *pvp = nvfx->vertprog;
+       struct nvfx_vertex_program* vp;
        struct pipe_resource *constbuf;
-       struct pipe_transfer *transfer;
        boolean upload_code = FALSE, upload_data = FALSE;
        int i;
 
        if (nvfx->render_mode == HW) {
-               vp = nvfx->vertprog;
-               constbuf = nvfx->constbuf[PIPE_SHADER_VERTEX];
+               nvfx->fallback_swtnl &= ~NVFX_NEW_VERTPROG;
+               vp = pvp->vp;
+
+               if(!vp) {
+                       vp = nvfx_vertprog_translate(nvfx, &pvp->pipe, &pvp->info);
+                       if(!vp)
+                               vp = NVFX_VP_FAILED;
+                       pvp->vp = vp;
+               }
 
-               if ((nvfx->dirty & NVFX_NEW_UCP) ||
-                   memcmp(&nvfx->clip, &vp->ucp, sizeof(vp->ucp))) {
-                       nvfx_vertprog_destroy(nvfx, vp);
-                       memcpy(&vp->ucp, &nvfx->clip, sizeof(vp->ucp));
+               if(vp == NVFX_VP_FAILED) {
+                       nvfx->fallback_swtnl |= NVFX_NEW_VERTPROG;
+                       return FALSE;
                }
+
+               constbuf = nvfx->constbuf[PIPE_SHADER_VERTEX];
        } else {
-               vp = nvfx->swtnl.vertprog;
+               vp = pvp->draw_vp;
+               if(!vp)
+               {
+                       pvp->draw_vp = vp = nvfx_vertprog_translate_draw_vp(nvfx, pvp);
+                       if(!vp) {
+                               _debug_printf("Error: unable to create a swtnl passthrough vertex shader: aborting.");
+                               abort();
+                       }
+               }
                constbuf = NULL;
        }
 
-       /* Translate TGSI shader into hw bytecode */
-       if (vp->translated)
-               goto check_gpu_resources;
-
-       nvfx->fallback_swtnl &= ~NVFX_NEW_VERTPROG;
-               nvfx_vertprog_translate(nvfx, vp);
-       if (!vp->translated) {
-               nvfx->fallback_swtnl |= NVFX_NEW_VERTPROG;
-                       return FALSE;
-       }
+       nvfx->hw_vertprog = vp;
 
-check_gpu_resources:
        /* Allocate hw vtxprog exec slots */
        if (!vp->exec) {
                struct nouveau_resource *heap = nvfx->screen->vp_exec_heap;
-               struct nouveau_stateobj *so;
                uint vplen = vp->nr_insns;
 
                if (nouveau_resource_alloc(heap, vplen, vp, &vp->exec)) {
@@ -884,21 +1235,12 @@ check_gpu_resources:
                        }
 
                        if (nouveau_resource_alloc(heap, vplen, vp, &vp->exec))
-                               assert(0);
-               }
-
-               so = so_new(3, 4, 0);
-               so_method(so, eng3d, NV34TCL_VP_START_FROM_ID, 1);
-               so_data  (so, vp->exec->start);
-               if(nvfx->is_nv4x) {
-                       so_method(so, eng3d, NV40TCL_VP_ATTRIB_EN, 2);
-                       so_data  (so, vp->ir);
-                       so_data  (so, vp->or);
+                       {
+                               debug_printf("Vertex shader too long: %u instructions\n", vplen);
+                               nvfx->fallback_swtnl |= NVFX_NEW_VERTPROG;
+                               return FALSE;
+                       }
                }
-               so_method(so, eng3d,  NV34TCL_VP_CLIP_PLANES_ENABLE, 1);
-               so_data  (so, vp->clip_ctrl);
-               so_ref(so, &vp->so);
-               so_ref(NULL, &so);
 
                upload_code = TRUE;
        }
@@ -916,9 +1258,15 @@ check_gpu_resources:
                        }
 
                        if (nouveau_resource_alloc(heap, vp->nr_consts, vp, &vp->data))
-                               assert(0);
+                        {
+                                debug_printf("Vertex shader uses too many constants: %u constants\n", vp->nr_consts);
+                                nvfx->fallback_swtnl |= NVFX_NEW_VERTPROG;
+                                return FALSE;
+                        }
                }
 
+               //printf("start at %u nc %u\n", vp->data->start, vp->nr_consts);
+
                /*XXX: handle this some day */
                assert(vp->data->start >= vp->data_start_min);
 
@@ -931,44 +1279,70 @@ check_gpu_resources:
         * fixup offsets and register IDs.
         */
        if (vp->exec_start != vp->exec->start) {
-               for (i = 0; i < vp->nr_insns; i++) {
-                       struct nvfx_vertex_program_exec *vpi = &vp->insns[i];
+               //printf("vp_relocs %u -> %u\n", vp->exec_start, vp->exec->start);
+               for(unsigned i = 0; i < vp->branch_relocs.size; i += sizeof(struct nvfx_relocation))
+               {
+                       struct nvfx_relocation* reloc = (struct nvfx_relocation*)((char*)vp->branch_relocs.data + i);
+                       uint32_t* hw = vp->insns[reloc->location].data;
+                       unsigned target = vp->exec->start + reloc->target;
 
-                       if (vpi->has_branch_offset) {
-                               assert(0);
+                       //debug_printf("vp_reloc hw %u -> hw %u\n", reloc->location, target);
+
+                       if(!nvfx->is_nv4x)
+                       {
+                               hw[2] &=~ NV30_VP_INST_IADDR_MASK;
+                               hw[2] |= (target & 0x1ff) << NV30_VP_INST_IADDR_SHIFT;
+                       }
+                       else
+                       {
+                               hw[3] &=~ NV40_VP_INST_IADDRL_MASK;
+                               hw[3] |= (target & 7) << NV40_VP_INST_IADDRL_SHIFT;
+
+                               hw[2] &=~ NV40_VP_INST_IADDRH_MASK;
+                               hw[2] |= ((target >> 3) & 0x3f) << NV40_VP_INST_IADDRH_SHIFT;
                        }
                }
 
                vp->exec_start = vp->exec->start;
        }
 
-       if (vp->nr_consts && vp->data_start != vp->data->start) {
-               for (i = 0; i < vp->nr_insns; i++) {
-                       struct nvfx_vertex_program_exec *vpi = &vp->insns[i];
+       if (vp->data_start != vp->data->start) {
+               for(unsigned i = 0; i < vp->const_relocs.size; i += sizeof(struct nvfx_relocation))
+               {
+                       struct nvfx_relocation* reloc = (struct nvfx_relocation*)((char*)vp->const_relocs.data + i);
+                       struct nvfx_vertex_program_exec *vpi = &vp->insns[reloc->location];
 
-                       if (vpi->const_index >= 0) {
-                               vpi->data[1] &= ~NVFX_VP(INST_CONST_SRC_MASK);
-                               vpi->data[1] |=
-                                       (vpi->const_index + vp->data->start) <<
-                                       NVFX_VP(INST_CONST_SRC_SHIFT);
+                       //printf("reloc %i to %i + %i\n", reloc->location, vp->data->start, reloc->target);
 
-                       }
+                       vpi->data[1] &= ~NVFX_VP(INST_CONST_SRC_MASK);
+                       vpi->data[1] |=
+                                       (reloc->target + vp->data->start) <<
+                                       NVFX_VP(INST_CONST_SRC_SHIFT);
                }
 
                vp->data_start = vp->data->start;
+               upload_code = TRUE;
        }
 
        /* Update + Upload constant values */
        if (vp->nr_consts) {
                float *map = NULL;
 
-               if (constbuf) {
-                       map = pipe_buffer_map(pipe, constbuf,
-                                             PIPE_TRANSFER_READ,
-                                             &transfer);
+               if (constbuf)
+                       map = (float*)nvfx_buffer(constbuf)->data;
+
+               /*
+                * WAIT_RING(chan, 512 * 6);
+               for (i = 0; i < 512; i++) {
+                       float v[4] = {0.1, 0,2, 0.3, 0.4};
+                       OUT_RING(chan, RING_3D(NV30_3D_VP_UPLOAD_CONST_ID, 5));
+                       OUT_RING(chan, i);
+                       OUT_RINGp(chan, (uint32_t *)v, 4);
+                       printf("frob %i\n", i);
                }
+               */
 
-               for (i = 0; i < vp->nr_consts; i++) {
+               for (i = nvfx->use_vp_clipping ? 6 : 0; i < vp->nr_consts; i++) {
                        struct nvfx_vertex_program_data *vpd = &vp->consts[i];
 
                        if (vpd->index >= 0) {
@@ -980,72 +1354,100 @@ check_gpu_resources:
                                       4 * sizeof(float));
                        }
 
-                       BEGIN_RING(chan, eng3d, NV34TCL_VP_UPLOAD_CONST_ID, 5);
-                       OUT_RING  (chan, i + vp->data->start);
-                       OUT_RINGp (chan, (uint32_t *)vpd->value, 4);
-               }
+                       //printf("upload into %i + %i: %f %f %f %f\n", vp->data->start, i, vpd->value[0], vpd->value[1], vpd->value[2], vpd->value[3]);
 
-               if (constbuf)
-                       pipe_buffer_unmap(pipe, constbuf, transfer);
+                       BEGIN_RING(chan, eng3d, NV30_3D_VP_UPLOAD_CONST_ID, 5);
+                       OUT_RING(chan, i + vp->data->start);
+                       OUT_RINGp(chan, (uint32_t *)vpd->value, 4);
+               }
        }
 
        /* Upload vtxprog */
        if (upload_code) {
-#if 0
+               BEGIN_RING(chan, eng3d, NV30_3D_VP_UPLOAD_FROM_ID, 1);
+               OUT_RING(chan, vp->exec->start);
                for (i = 0; i < vp->nr_insns; i++) {
-                       NOUVEAU_MSG("VP %d: 0x%08x\n", i, vp->insns[i].data[0]);
-                       NOUVEAU_MSG("VP %d: 0x%08x\n", i, vp->insns[i].data[1]);
-                       NOUVEAU_MSG("VP %d: 0x%08x\n", i, vp->insns[i].data[2]);
-                       NOUVEAU_MSG("VP %d: 0x%08x\n", i, vp->insns[i].data[3]);
-               }
-#endif
-               BEGIN_RING(chan, eng3d, NV34TCL_VP_UPLOAD_FROM_ID, 1);
-               OUT_RING  (chan, vp->exec->start);
-               for (i = 0; i < vp->nr_insns; i++) {
-                       BEGIN_RING(chan, eng3d, NV34TCL_VP_UPLOAD_INST(0), 4);
-                       OUT_RINGp (chan, vp->insns[i].data, 4);
+                       BEGIN_RING(chan, eng3d, NV30_3D_VP_UPLOAD_INST(0), 4);
+                       //printf("%08x %08x %08x %08x\n", vp->insns[i].data[0], vp->insns[i].data[1], vp->insns[i].data[2], vp->insns[i].data[3]);
+                       OUT_RINGp(chan, vp->insns[i].data, 4);
                }
+               vp->clip_nr = -1;
        }
 
-       if (vp->so != nvfx->state.hw[NVFX_STATE_VERTPROG]) {
-               so_ref(vp->so, &nvfx->state.hw[NVFX_STATE_VERTPROG]);
-               return TRUE;
+       if(nvfx->dirty & (NVFX_NEW_VERTPROG))
+       {
+               BEGIN_RING(chan, eng3d, NV30_3D_VP_START_FROM_ID, 1);
+               OUT_RING(chan, vp->exec->start);
+               if(nvfx->is_nv4x) {
+                       BEGIN_RING(chan, eng3d, NV40_3D_VP_ATTRIB_EN, 1);
+                       OUT_RING(chan, vp->ir);
+               }
        }
 
-       return FALSE;
+       return TRUE;
 }
 
 void
 nvfx_vertprog_destroy(struct nvfx_context *nvfx, struct nvfx_vertex_program *vp)
 {
-       vp->translated = FALSE;
-
-       if (vp->nr_insns) {
+       if (vp->nr_insns)
                FREE(vp->insns);
-               vp->insns = NULL;
-               vp->nr_insns = 0;
-       }
 
-       if (vp->nr_consts) {
+       if (vp->nr_consts)
                FREE(vp->consts);
-               vp->consts = NULL;
-               vp->nr_consts = 0;
-       }
 
        nouveau_resource_free(&vp->exec);
-       vp->exec_start = 0;
        nouveau_resource_free(&vp->data);
-       vp->data_start = 0;
-       vp->data_start_min = 0;
 
-       vp->ir = vp->or = vp->clip_ctrl = 0;
-       so_ref(NULL, &vp->so);
+       util_dynarray_fini(&vp->branch_relocs);
+       util_dynarray_fini(&vp->const_relocs);
+       FREE(vp);
 }
 
-struct nvfx_state_entry nvfx_state_vertprog = {
-       .validate = nvfx_vertprog_validate,
-       .dirty = {
-               .pipe = NVFX_NEW_VERTPROG | NVFX_NEW_UCP,
-               .hw = NVFX_STATE_VERTPROG,
-       }
-};
+static void *
+nvfx_vp_state_create(struct pipe_context *pipe, const struct pipe_shader_state *cso)
+{
+        struct nvfx_pipe_vertex_program *pvp;
+
+        pvp = CALLOC(1, sizeof(struct nvfx_pipe_vertex_program));
+        pvp->pipe.tokens = tgsi_dup_tokens(cso->tokens);
+        tgsi_scan_shader(pvp->pipe.tokens, &pvp->info);
+        pvp->draw_elements = MAX2(1, MIN2(pvp->info.num_outputs, 16));
+        pvp->draw_no_elements = pvp->info.num_outputs == 0;
+
+        return (void *)pvp;
+}
+
+static void
+nvfx_vp_state_bind(struct pipe_context *pipe, void *hwcso)
+{
+        struct nvfx_context *nvfx = nvfx_context(pipe);
+
+        nvfx->vertprog = hwcso;
+        nvfx->dirty |= NVFX_NEW_VERTPROG;
+        nvfx->draw_dirty |= NVFX_NEW_VERTPROG;
+}
+
+static void
+nvfx_vp_state_delete(struct pipe_context *pipe, void *hwcso)
+{
+       struct nvfx_context *nvfx = nvfx_context(pipe);
+       struct nvfx_pipe_vertex_program *pvp = hwcso;
+
+       if(pvp->draw_vs)
+               draw_delete_vertex_shader(nvfx->draw, pvp->draw_vs);
+       if(pvp->vp && pvp->vp != NVFX_VP_FAILED)
+               nvfx_vertprog_destroy(nvfx, pvp->vp);
+       if(pvp->draw_vp)
+               nvfx_vertprog_destroy(nvfx, pvp->draw_vp);
+       FREE((void*)pvp->pipe.tokens);
+       FREE(pvp);
+}
+
+void
+nvfx_init_vertprog_functions(struct nvfx_context *nvfx)
+{
+        nvfx->pipe.create_vs_state = nvfx_vp_state_create;
+        nvfx->pipe.bind_vs_state = nvfx_vp_state_bind;
+        nvfx->pipe.delete_vs_state = nvfx_vp_state_delete;
+}