From: Marek Olšák Date: Sat, 6 Jul 2019 03:22:33 +0000 (-0400) Subject: radeonsi/gfx10: don't use MALLOC for outputs X-Git-Tag: upstream/19.3.0~3963 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=2683347ba0394a5e17223aab2f685775f87e1fac;p=platform%2Fupstream%2Fmesa.git radeonsi/gfx10: don't use MALLOC for outputs Acked-by: Pierre-Eric Pelloux-Prayer Reviewed-by: Samuel Pitoiset --- diff --git a/src/gallium/drivers/radeonsi/gfx10_shader_ngg.c b/src/gallium/drivers/radeonsi/gfx10_shader_ngg.c index 74be6df..e074449 100644 --- a/src/gallium/drivers/radeonsi/gfx10_shader_ngg.c +++ b/src/gallium/drivers/radeonsi/gfx10_shader_ngg.c @@ -521,7 +521,7 @@ void gfx10_emit_ngg_epilogue(struct ac_shader_abi *abi, struct si_shader_context *ctx = si_shader_context_from_abi(abi); struct si_shader_selector *sel = ctx->shader->selector; struct tgsi_shader_info *info = &sel->info; - struct si_shader_output_values *outputs = NULL; + struct si_shader_output_values outputs[PIPE_MAX_SHADER_OUTPUTS]; LLVMBuilderRef builder = ctx->ac.builder; struct lp_build_if_state if_state; LLVMValueRef tmp, tmp2; @@ -529,8 +529,6 @@ void gfx10_emit_ngg_epilogue(struct ac_shader_abi *abi, assert(!ctx->shader->is_gs_copy_shader); assert(info->num_outputs <= max_outputs); - outputs = MALLOC((info->num_outputs + 1) * sizeof(outputs[0])); - LLVMValueRef vertex_ptr = NULL; if (sel->so.num_outputs) @@ -756,8 +754,6 @@ void gfx10_emit_ngg_epilogue(struct ac_shader_abi *abi, si_llvm_export_vs(ctx, outputs, i); } lp_build_endif(&if_state); - - FREE(outputs); } static LLVMValueRef @@ -1206,8 +1202,7 @@ void gfx10_ngg_gs_emit_epilogue(struct si_shader_context *ctx) tmp = LLVMBuildICmp(builder, LLVMIntULT, tid, vertlive_scan.result_reduce, ""); ac_build_ifcc(&ctx->ac, tmp, 5145); { - struct si_shader_output_values *outputs = NULL; - outputs = MALLOC(info->num_outputs * sizeof(outputs[0])); + struct si_shader_output_values outputs[PIPE_MAX_SHADER_OUTPUTS]; tmp = ngg_gs_vertex_ptr(ctx, tid); LLVMValueRef gep_idx[3] = { @@ -1237,8 +1232,6 @@ void gfx10_ngg_gs_emit_epilogue(struct si_shader_context *ctx) } si_llvm_export_vs(ctx, outputs, info->num_outputs); - - FREE(outputs); } ac_build_endif(&ctx->ac, 5145); }