[Mesa-dev] [PATCH 1/5] i965: wrap assert param in #ifndef NDEBUG

Iago Toral itoral at igalia.com
Tue Oct 4 08:33:49 UTC 2016


The series is:

Reviewed-by: Iago Toral Quiroga <itoral at igalia.com>

On Tue, 2016-10-04 at 11:15 +1100, Timothy Arceri wrote:
> This fixes an unused variable warning on release builds.
> ---
>  src/mesa/drivers/dri/i965/brw_fs.cpp | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
> b/src/mesa/drivers/dri/i965/brw_fs.cpp
> index dc000d9..ccf311d 100644
> --- a/src/mesa/drivers/dri/i965/brw_fs.cpp
> +++ b/src/mesa/drivers/dri/i965/brw_fs.cpp
> @@ -5866,7 +5866,9 @@ fs_visitor::allocate_registers(bool
> allow_spilling)
>     schedule_instructions(SCHEDULE_POST);
>  
>     if (last_scratch > 0) {
> +#ifndef NDEBUG
>        unsigned max_scratch_size = 2 * 1024 * 1024;
> +#endif
>  
>        prog_data->total_scratch = brw_get_scratch_size(last_scratch);
>  
> @@ -5884,7 +5886,9 @@ fs_visitor::allocate_registers(bool
> allow_spilling)
>               * size linearly with a range of [1kB, 12kB] and 1kB
> granularity.
>               */
>              prog_data->total_scratch = ALIGN(last_scratch, 1024);
> +#ifndef NDEBUG
>              max_scratch_size = 12 * 1024;
> +#endif
>           }
>        }
>  


More information about the mesa-dev mailing list