Mesa (master): r300/compiler: Remove inst_offset from r500_fragment_program_code

Nicolai Hähnle nh at kemper.freedesktop.org
Thu Jul 30 21:44:44 UTC 2009


Module: Mesa
Branch: master
Commit: cab62aa28f5ccdf7ca185ac965b852e2318816f3
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=cab62aa28f5ccdf7ca185ac965b852e2318816f3

Author: Nicolai Hähnle <nhaehnle at gmail.com>
Date:   Thu Jul 30 22:26:02 2009 +0200

r300/compiler: Remove inst_offset from r500_fragment_program_code

The field is not used, and in any case it would be more interesting to
manipulate from *outside* the compiler if we ever wanted to load several
fragment programs at the same time or something.

Signed-off-by: Nicolai Hähnle <nhaehnle at gmail.com>

---

 .../drivers/dri/r300/compiler/r500_fragprog_emit.c |    1 -
 src/mesa/drivers/dri/r300/compiler/radeon_code.h   |    3 +--
 src/mesa/drivers/dri/r300/r300_state.c             |    6 +++---
 3 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/src/mesa/drivers/dri/r300/compiler/r500_fragprog_emit.c b/src/mesa/drivers/dri/r300/compiler/r500_fragprog_emit.c
index 3a52721..d694725 100644
--- a/src/mesa/drivers/dri/r300/compiler/r500_fragprog_emit.c
+++ b/src/mesa/drivers/dri/r300/compiler/r500_fragprog_emit.c
@@ -279,7 +279,6 @@ void r500BuildFragmentProgramHwCode(struct r300_fragment_program_compiler *compi
 
 	_mesa_bzero(code, sizeof(*code));
 	code->max_temp_idx = 1;
-	code->inst_offset = 0;
 	code->inst_end = -1;
 
 	radeonPairProgram(compiler, &pair_handler, compiler);
diff --git a/src/mesa/drivers/dri/r300/compiler/radeon_code.h b/src/mesa/drivers/dri/r300/compiler/radeon_code.h
index 6f5bc28..0806fb1 100644
--- a/src/mesa/drivers/dri/r300/compiler/radeon_code.h
+++ b/src/mesa/drivers/dri/r300/compiler/radeon_code.h
@@ -160,8 +160,7 @@ struct r500_fragment_program_code {
 		uint32_t inst5;
 	} inst[R500_PFS_MAX_INST];
 
-	int inst_offset;
-	int inst_end;
+	int inst_end; /* Number of instructions - 1; also, last instruction to be executed */
 
 	int max_temp_idx;
 };
diff --git a/src/mesa/drivers/dri/r300/r300_state.c b/src/mesa/drivers/dri/r300/r300_state.c
index 1ac1426..b868b62 100644
--- a/src/mesa/drivers/dri/r300/r300_state.c
+++ b/src/mesa/drivers/dri/r300/r300_state.c
@@ -2103,13 +2103,13 @@ static void r500SetupPixelShader(GLcontext *ctx)
 	rmesa->hw.fp.cmd[R500_FP_PIXSIZE] = code->max_temp_idx;
 
 	rmesa->hw.fp.cmd[R500_FP_CODE_ADDR] =
-	    R500_US_CODE_START_ADDR(code->inst_offset) |
+	    R500_US_CODE_START_ADDR(0) |
 	    R500_US_CODE_END_ADDR(code->inst_end);
 	rmesa->hw.fp.cmd[R500_FP_CODE_RANGE] =
-	    R500_US_CODE_RANGE_ADDR(code->inst_offset) |
+	    R500_US_CODE_RANGE_ADDR(0) |
 	    R500_US_CODE_RANGE_SIZE(code->inst_end);
 	rmesa->hw.fp.cmd[R500_FP_CODE_OFFSET] =
-	    R500_US_CODE_OFFSET_ADDR(0); /* FIXME when we add flow control */
+	    R500_US_CODE_OFFSET_ADDR(0);
 
 	R300_STATECHANGE(rmesa, r500fp);
 	/* Emit our shader... */




More information about the mesa-commit mailing list