[Mesa-dev] [PATCH 3/4] i965: Silence compiler warning about promoted_constants.

Lionel Landwerlin lionel.g.landwerlin at intel.com
Tue Mar 13 18:09:59 UTC 2018


On 13/03/18 17:38, Eric Anholt wrote:
> We only have a cfg != NULL if we went through one of the paths that set
> it, but my compiler doesn't figure that out.
>
> Cc: Jason Ekstrand <jason.ekstrand at intel.com>

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>

> Fixes: 6411defdcd6f ("intel/cs: Re-run final NIR optimizations for each SIMD size")
> ---
>   src/intel/compiler/brw_fs.cpp | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/intel/compiler/brw_fs.cpp b/src/intel/compiler/brw_fs.cpp
> index 422eedcf0af1..8529fda6f091 100644
> --- a/src/intel/compiler/brw_fs.cpp
> +++ b/src/intel/compiler/brw_fs.cpp
> @@ -7220,7 +7220,7 @@ brw_compile_cs(const struct brw_compiler *compiler, void *log_data,
>      fs_visitor *v8 = NULL, *v16 = NULL, *v32 = NULL;
>      cfg_t *cfg = NULL;
>      const char *fail_msg = NULL;
> -   unsigned promoted_constants;
> +   unsigned promoted_constants = 0;
>   
>      /* Now the main event: Visit the shader IR and generate our CS IR for it.
>       */




More information about the mesa-dev mailing list