From 47e16cab5ee949ba5326b6f0234a184921932e89 Mon Sep 17 00:00:00 2001 From: Alyssa Rosenzweig Date: Thu, 16 Jan 2025 09:36:04 -0500 Subject: [PATCH] nir/lower_printf: drop default max buffer size no uses and it doesn't make sense. Signed-off-by: Alyssa Rosenzweig Reviewed-by: Jesse Natalie Reviewed-by: Lionel Landwerlin Part-of: --- src/compiler/nir/nir_lower_printf.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/compiler/nir/nir_lower_printf.c b/src/compiler/nir/nir_lower_printf.c index e6ec66f0912..14cb991aa0f 100644 --- a/src/compiler/nir/nir_lower_printf.c +++ b/src/compiler/nir/nir_lower_printf.c @@ -108,10 +108,8 @@ lower_printf_intrin(nir_builder *b, nir_intrinsic_instr *prntf, void *_options) .atomic_op = nir_atomic_op_iadd); /* Check if we're still in-bounds */ - const unsigned default_buffer_size = 1024 * 1024; - unsigned buffer_size = (options && options->max_buffer_size) ? options->max_buffer_size : default_buffer_size; int max_valid_offset = - buffer_size - args_size - fmt_str_id_size - counter_size; + options->max_buffer_size - args_size - fmt_str_id_size - counter_size; assert(max_valid_offset >= 0 && "printf buffer must be sufficiently large"); nir_push_if(b, nir_ult_imm(b, offset, max_valid_offset));