This patch removes the following field from vec4_generator, since it is not used:
- struct brw_vs_compile *c And changes the following field: - struct gl_vertex_program *vp => struct gl_program *prog With these changes, vec4_generator no longer refers to any VS-specific data structures. This will pave the way for re-using it for geometry shaders. Reviewed-by: Jordan Justen <jordan.l.jus...@intel.com> v2: Use the name "prog" rather than "p". --- src/mesa/drivers/dri/i965/brw_vec4.cpp | 2 +- src/mesa/drivers/dri/i965/brw_vec4.h | 5 ++--- src/mesa/drivers/dri/i965/brw_vec4_emit.cpp | 9 ++++----- 3 files changed, 7 insertions(+), 9 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_vec4.cpp b/src/mesa/drivers/dri/i965/brw_vec4.cpp index 9026501..b366036 100644 --- a/src/mesa/drivers/dri/i965/brw_vec4.cpp +++ b/src/mesa/drivers/dri/i965/brw_vec4.cpp @@ -1526,7 +1526,7 @@ brw_vs_emit(struct brw_context *brw, return NULL; } - vec4_generator g(brw, c, prog, mem_ctx); + vec4_generator g(brw, prog, &c->vp->program.Base, mem_ctx); const unsigned *generated =g.generate_assembly(&v.instructions, final_assembly_size); diff --git a/src/mesa/drivers/dri/i965/brw_vec4.h b/src/mesa/drivers/dri/i965/brw_vec4.h index dca6a55..0c9fb70 100644 --- a/src/mesa/drivers/dri/i965/brw_vec4.h +++ b/src/mesa/drivers/dri/i965/brw_vec4.h @@ -523,8 +523,8 @@ class vec4_generator { public: vec4_generator(struct brw_context *brw, - struct brw_vs_compile *c, struct gl_shader_program *shader_prog, + struct gl_program *prog, void *mem_ctx); ~vec4_generator(); @@ -579,11 +579,10 @@ private: struct gl_context *ctx; struct brw_compile *p; - struct brw_vs_compile *c; struct gl_shader_program *shader_prog; struct gl_shader *shader; - const struct gl_vertex_program *vp; + const struct gl_program *prog; void *mem_ctx; }; diff --git a/src/mesa/drivers/dri/i965/brw_vec4_emit.cpp b/src/mesa/drivers/dri/i965/brw_vec4_emit.cpp index 1c99d6a..fc00cb3 100644 --- a/src/mesa/drivers/dri/i965/brw_vec4_emit.cpp +++ b/src/mesa/drivers/dri/i965/brw_vec4_emit.cpp @@ -133,13 +133,12 @@ vec4_instruction::get_src(int i) } vec4_generator::vec4_generator(struct brw_context *brw, - struct brw_vs_compile *c, struct gl_shader_program *shader_prog, + struct gl_program *prog, void *mem_ctx) - : brw(brw), c(c), shader_prog(shader_prog), mem_ctx(mem_ctx) + : brw(brw), shader_prog(shader_prog), prog(prog), mem_ctx(mem_ctx) { intel = &brw->intel; - vp = &c->vp->program; shader = shader_prog ? shader_prog->_LinkedShaders[MESA_SHADER_VERTEX] : NULL; @@ -699,7 +698,7 @@ vec4_generator::generate_code(exec_list *instructions) if (shader) { printf("Native code for vertex shader %d:\n", shader_prog->Name); } else { - printf("Native code for vertex program %d:\n", c->vp->program.Base.Id); + printf("Native code for vertex program %d:\n", prog->Id); } } @@ -717,7 +716,7 @@ vec4_generator::generate_code(exec_list *instructions) } else { const prog_instruction *vpi; vpi = (const prog_instruction *) inst->ir; - printf("%d: ", (int)(vpi - vp->Base.Instructions)); + printf("%d: ", (int)(vpi - prog->Instructions)); _mesa_fprint_instruction_opt(stdout, vpi, 0, PROG_PRINT_DEBUG, NULL); } -- 1.8.2.1 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev