<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Tue, Nov 13, 2018 at 6:35 PM Eric Engestrom <<a href="mailto:eric.engestrom@intel.com">eric.engestrom@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tuesday, 2018-11-13 14:19:30 +0200, <a href="mailto:asimiklit.work@gmail.com" target="_blank">asimiklit.work@gmail.com</a> wrote:<br>
> From: Andrii Simiklit <<a href="mailto:andrii.simiklit@globallogic.com" target="_blank">andrii.simiklit@globallogic.com</a>><br>
> <br>
> 1. brw_pipe_control.c:311:34: warning:<br>
>     unused variable ‘devinfo’<br>
> 2. brw_program_binary.c:209:19: warning:<br>
>     unused variable ‘gen_size’<br>
> 3. brw_program_binary.c:216:19: warning:<br>
>     unused variable ‘nir_size’<br>
> <br>
> v2: Changes for unreproducible issues were removed<br>
> <br>
> Signed-off-by: Andrii Simiklit <<a href="mailto:andrii.simiklit@globallogic.com" target="_blank">andrii.simiklit@globallogic.com</a>><br>
> ---<br>
>  src/mesa/drivers/dri/i965/brw_pipe_control.c   | 2 +-<br>
>  src/mesa/drivers/dri/i965/brw_program_binary.c | 4 ++--<br>
>  2 files changed, 3 insertions(+), 3 deletions(-)<br>
> <br>
> diff --git a/src/mesa/drivers/dri/i965/brw_pipe_control.c b/src/mesa/drivers/dri/i965/brw_pipe_control.c<br>
> index 122ac26070..a3f521b5ae 100644<br>
> --- a/src/mesa/drivers/dri/i965/brw_pipe_control.c<br>
> +++ b/src/mesa/drivers/dri/i965/brw_pipe_control.c<br>
> @@ -308,7 +308,7 @@ brw_emit_depth_stall_flushes(struct brw_context *brw)<br>
>  void<br>
>  gen7_emit_vs_workaround_flush(struct brw_context *brw)<br>
>  {<br>
> -   const struct gen_device_info *devinfo = &brw->screen->devinfo;<br>
> +   MAYBE_UNUSED const struct gen_device_info *devinfo = &brw->screen->devinfo;<br>
>  <br>
>     assert(devinfo->gen == 7);<br>
<br>
This could've just been folded into the assert, but this works.<br>
<br>
Patches 1-3 are:<br>
Reviewed-by: Eric Engestrom <<a href="mailto:eric.engestrom@intel.com" target="_blank">eric.engestrom@intel.com</a>><br></blockquote><div><br></div><div>Thanks a lot for review.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I assume you want me to push them for you?<br></blockquote><div><br></div><div>Yes, it would be great. Thanks )<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
>     brw_emit_pipe_control_write(brw,<br>
> diff --git a/src/mesa/drivers/dri/i965/brw_program_binary.c b/src/mesa/drivers/dri/i965/brw_program_binary.c<br>
> index db03332241..1298d9e765 100644<br>
> --- a/src/mesa/drivers/dri/i965/brw_program_binary.c<br>
> +++ b/src/mesa/drivers/dri/i965/brw_program_binary.c<br>
> @@ -206,14 +206,14 @@ brw_program_deserialize_driver_blob(struct gl_context *ctx,<br>
>           break;<br>
>        switch ((enum driver_cache_blob_part)part_type) {<br>
>        case GEN_PART: {<br>
> -         uint32_t gen_size = blob_read_uint32(&reader);<br>
> +         MAYBE_UNUSED uint32_t gen_size = blob_read_uint32(&reader);<br>
>           assert(!reader.overrun &&<br>
>                  (uintptr_t)(reader.end - reader.current) > gen_size);<br>
>           deserialize_gen_program(&reader, ctx, prog, stage);<br>
>           break;<br>
>        }<br>
>        case NIR_PART: {<br>
> -         uint32_t nir_size = blob_read_uint32(&reader);<br>
> +         MAYBE_UNUSED uint32_t nir_size = blob_read_uint32(&reader);<br>
>           assert(!reader.overrun &&<br>
>                  (uintptr_t)(reader.end - reader.current) > nir_size);<br>
>           const struct nir_shader_compiler_options *options =<br>
> -- <br>
> 2.17.1<br>
> <br>
</blockquote></div></div>