[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