[Mesa-dev] [PATCH] anv: fix warning about unused width variable.

Eduardo Lima Mitev elima at igalia.com
Fri Feb 19 10:08:50 UTC 2016


On 02/17/2016 03:23 AM, Dave Airlie wrote:
> From: Dave Airlie <airlied at redhat.com>
> 
> We don't use width outside the debug clause here.
> ---

This issue is still valid, but patch needs rebasing. There are at least
two instances of this problem in gen7_pack.h, maybe more in the rest of
pack header generators.

Eduardo

>  src/vulkan/gen_pack_header.py | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/src/vulkan/gen_pack_header.py b/src/vulkan/gen_pack_header.py
> index 3cabb58..75c4f26 100755
> --- a/src/vulkan/gen_pack_header.py
> +++ b/src/vulkan/gen_pack_header.py
> @@ -62,11 +62,10 @@ __gen_mbo(uint32_t start, uint32_t end)
>  static inline uint64_t
>  __gen_uint(uint64_t v, uint32_t start, uint32_t end)
>  {
> -   const int width = end - start + 1;
> -
>     __gen_validate_value(v);
>  
>  #if DEBUG
> +   const int width = end - start + 1;
>     if (width < 64) {
>        const uint64_t max = (1ull << width) - 1;
>        assert(v <= max);
> 



More information about the mesa-dev mailing list