radeonsi/gfx10: update comments and remove invalid TODOs

Acked-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com>

diff --git a/src/gallium/drivers/radeonsi/gfx10_shader_ngg.c b/src/gallium/drivers/radeonsi/gfx10_shader_ngg.c
index afb9ad3f6ee2de61118faaea61fd2506aee17550..4d19f714e6c00c92e42c2c9987569809b28a267d 100644 (file)
--- a/src/gallium/drivers/radeonsi/gfx10_shader_ngg.c
+++ b/src/gallium/drivers/radeonsi/gfx10_shader_ngg.c
@@ -640,7 +640,6 @@ void gfx10_emit_ngg_epilogue(struct ac_shader_abi *abi,
                                      ngg_get_vtx_cnt(ctx), ngg_get_prim_cnt(ctx));
 
        /* Update query buffer */
-       /* TODO: this won't catch 96-bit clear_buffer via transform feedback. */
        if (ctx->screen->use_ngg_streamout &&
            !info->properties[TGSI_PROPERTY_VS_BLIT_SGPRS_AMD]) {
                tmp = si_unpack_param(ctx, ctx->vs_state_bits, 6, 1);
@@ -675,15 +674,7 @@ void gfx10_emit_ngg_epilogue(struct ac_shader_abi *abi,
                ac_build_endif(&ctx->ac, 5029);
        }
 
-       /* Build the primitive export.
-        *
-        * For the first version, we will always build up all three indices
-        * independent of the primitive type. The additional garbage data
-        * shouldn't hurt.
-        *
-        * TODO: culling depends on the primitive type, so can have some
-        * interaction here.
-        */
+       /* Build the primitive export. */
        ac_build_ifcc(&ctx->ac, is_gs_thread, 6001);
        {
                struct ac_ngg_prim prim = {};
@@ -1093,8 +1084,6 @@ void gfx10_ngg_gs_emit_epilogue(struct si_shader_context *ctx)
                ac_build_endif(&ctx->ac, 5109);
        }
 
-       /* TODO: culling */
-
        /* Determine vertex liveness. */
        LLVMValueRef vertliveptr = ac_build_alloca(&ctx->ac, ctx->ac.i1, "vertexlive");