Mesa (master): radeonsi: Remove some stale XXX / FIXME comments

Michel Dänzer daenzer at kemper.freedesktop.org
Thu Dec 5 04:52:37 UTC 2013


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

Author: Michel Dänzer <michel.daenzer at amd.com>
Date:   Wed Dec  4 13:37:07 2013 +0900

radeonsi: Remove some stale XXX / FIXME comments

Reviewed-by: Marek Olšák <marek.olsak at amd.com>
Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

---

 src/gallium/drivers/radeonsi/radeonsi_pipe.c   |    1 -
 src/gallium/drivers/radeonsi/radeonsi_shader.c |    5 +----
 2 files changed, 1 insertions(+), 5 deletions(-)

diff --git a/src/gallium/drivers/radeonsi/radeonsi_pipe.c b/src/gallium/drivers/radeonsi/radeonsi_pipe.c
index 0242a03..4aaa88f 100644
--- a/src/gallium/drivers/radeonsi/radeonsi_pipe.c
+++ b/src/gallium/drivers/radeonsi/radeonsi_pipe.c
@@ -392,7 +392,6 @@ static int r600_get_param(struct pipe_screen* pscreen, enum pipe_cap param)
 
 	/* Render targets. */
 	case PIPE_CAP_MAX_RENDER_TARGETS:
-		/* FIXME some r6xx are buggy and can only do 4 */
 		return 8;
 
 	case PIPE_CAP_MAX_VIEWPORTS:
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c b/src/gallium/drivers/radeonsi/radeonsi_shader.c
index 96cc1aa..88825bb 100644
--- a/src/gallium/drivers/radeonsi/radeonsi_shader.c
+++ b/src/gallium/drivers/radeonsi/radeonsi_shader.c
@@ -281,7 +281,6 @@ static void declare_input_fs(
 	attr_number = lp_build_const_int32(gallivm,
 					   shader->input[input_index].param_offset);
 
-	/* XXX: Handle all possible interpolation modes */
 	switch (decl->Interp.Interpolate) {
 	case TGSI_INTERPOLATE_COLOR:
 		if (si_shader_ctx->shader->key.ps.flatshade) {
@@ -875,7 +874,7 @@ static void si_llvm_emit_streamout(struct si_shader_context *shader)
 	lp_build_endif(&if_ctx);
 }
 
-/* XXX: This is partially implemented for VS only at this point.  It is not complete */
+
 static void si_llvm_emit_epilogue(struct lp_build_tgsi_context * bld_base)
 {
 	struct si_shader_context * si_shader_ctx = si_shader_context(bld_base);
@@ -1222,8 +1221,6 @@ handle_semantic:
 				   LLVMVoidTypeInContext(base->gallivm->context),
 				   last_args, 9);
 	}
-/* XXX: Look up what this function does */
-/*		ctx->shader->output[i].spi_sid = r600_spi_sid(&ctx->shader->output[i]);*/
 }
 
 static const struct lp_build_tgsi_action txf_action;




More information about the mesa-commit mailing list