[virglrenderer-devel] [PATCH 00/16] Fix warnings in gallium/auxiliary/*

[Gurchetan Singh]

I've always wondered why building Mesa (and Gallium) always spewed out
so many warnings.  Would it be possible to fix this in upstream Mesa,
and have the changes trickle down to here when update our Gallium
code?  I know we've forked Gallium, but is the fork hard enough that
this would not work (we essentially support a few more outdated TGSI
opcodes [TGSI_OPCODE_SCS etc.])?
On Thu, May 31, 2018 at 2:16 AM Gert Wollny <gert.wollny at collabora.com> wrote:
>
> Dear all,
>
> this patch series fixes all the warnings that gcc raises when compiling
> with -Wall -Wextra (mostly -Wsigne-compare and -Wunused parameters).
>
> Thanks for reviewing,
> Gert
>
> PS: I'm working on doing the same vor the remaining source files.
>
> Gert Wollny (16):
>   gallium/auxiliary/util: Add macro to silence unused parameter warning
>   gallium/aux/cso_cache.c: Fix warnings
>   gallium/aux/tgsi/tgsi_build.c: Fix warnings
>   gallium/aux/tgsi/tgsi_dump.c: Fix warnings
>   gallium/aux/tgsi/tgsi_parse.c: Fix warnings
>   gallium/aux/tgsi/tgsi_sanity.c: Fix warnings
>   gallium/aux/util/u_debug_refcnt.h: Fix warnings:
>   gallium/aux/tgsi/tgsi_text.c: Fix warnings
>   gallium/aux/util/u_surface.c: Fix warning
>   gallium/aux/tgsi/tgsi_ureg.*: Fix warnings
>   gallium/aux/tgsi/tgsi_util.c: Fix warnings
>   gallizm/aux/util/u_format.c: Fix warnings
>   gallium/aux/util/u_cpu_detect.h: Fix warning in u_cpu_detect.c
>   gallium/aux/tgsi/tgsi_scan.c: Fix warnings
>   gallium/auxiliary/util/u_debug.c: Fix warnings
>   gallium/auxiliary/util/u_debug.h: Fix "noreturn" warnings in debug
>     mode
>
>  src/gallium/auxiliary/cso_cache/cso_cache.c   |  6 ++++++
>  src/gallium/auxiliary/tgsi/tgsi_build.c       |  9 +++++++--
>  src/gallium/auxiliary/tgsi/tgsi_dump.c        |  2 +-
>  src/gallium/auxiliary/tgsi/tgsi_parse.c       |  9 +++++++--
>  src/gallium/auxiliary/tgsi/tgsi_sanity.c      |  4 ++--
>  src/gallium/auxiliary/tgsi/tgsi_scan.c        |  2 +-
>  src/gallium/auxiliary/tgsi/tgsi_text.c        |  8 ++++----
>  src/gallium/auxiliary/tgsi/tgsi_ureg.c        | 10 +++++++---
>  src/gallium/auxiliary/tgsi/tgsi_ureg.h        |  4 ++--
>  src/gallium/auxiliary/tgsi/tgsi_util.c        |  2 ++
>  src/gallium/auxiliary/util/u_cpu_detect.h     |  2 +-
>  src/gallium/auxiliary/util/u_debug.c          |  6 ++++++
>  src/gallium/auxiliary/util/u_debug.h          |  6 ++++--
>  src/gallium/auxiliary/util/u_debug_describe.c |  1 +
>  src/gallium/auxiliary/util/u_debug_refcnt.h   |  5 +++++
>  src/gallium/auxiliary/util/u_format.c         |  2 +-
>  src/gallium/auxiliary/util/u_surface.c        |  2 +-
>  17 files changed, 58 insertions(+), 22 deletions(-)
>
> --
> 2.16.1
>
> _______________________________________________
> virglrenderer-devel mailing list
> virglrenderer-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/virglrenderer-devel