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

Author: Marek Olšák <[email protected]>
Date:   Mon Dec 23 19:42:46 2019 -0500

radeonsi/gfx10: update comments and remove invalid TODOs

Acked-by: Pierre-Eric Pelloux-Prayer <[email protected]>

---

 src/gallium/drivers/radeonsi/gfx10_shader_ngg.c | 13 +------------
 1 file changed, 1 insertion(+), 12 deletions(-)

diff --git a/src/gallium/drivers/radeonsi/gfx10_shader_ngg.c 
b/src/gallium/drivers/radeonsi/gfx10_shader_ngg.c
index afb9ad3f6ee..4d19f714e6c 100644
--- 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");
 

_______________________________________________
mesa-commit mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-commit

Reply via email to