[Mesa-dev] [PATCH 00/10] Clean up most of the new gcc6 warnings

Rob Clark robdclark at gmail.com
Tue Feb 16 18:57:56 UTC 2016


From: Rob Clark <robclark at freedesktop.org>

gcc6 landed in rawhide, and all of a sudden building mesa got very
noisy.  This patchset cleans up most of the warnings (especially
the first one which shows up everywhere that #includes hash_table.h)

There are two remaining:

src/gallium/auxiliary/util/u_debug_stack.c: In function ‘debug_backtrace_capture’:
src/gallium/auxiliary/util/u_debug_stack.c:108:18: warning: calling ‘__builtin_frame_address’ with a nonzero argument is unsafe [-Wframe-address]
    frame_pointer = ((const void **)__builtin_frame_address(1));
    ~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Not sure there is anything we *can* do about that one.  And:

  src/gallium/auxiliary/util/u_debug_stack.c: In function ‘debug_backtrace_capture’:
  src/gallium/auxiliary/util/u_debug_stack.c:108:18: warning: calling ‘__builtin_frame_address’ with a nonzero argument is unsafe [-Wframe-address]
      frame_pointer = ((const void **)__builtin_frame_address(1));
      ~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Not sure there is anything we *can* do about that one.  And:

  src/mesa/main/get.c:473:18: warning: ‘extra_version_40’ defined but not used [-Wunused-const-variable]
   static const int extra_version_40[] = { EXTRA_VERSION_40, EXTRA_END };
                    ^~~~~~~~~~~~~~~~
  src/mesa/main/get.c:232:21: warning: ‘extra_ARB_compute_shader’ defined but not used [-Wunused-const-variable]
      static const int extra_##e[] = {  \
                       ^
  src/mesa/main/get.c:441:1: note: in expansion of macro ‘EXTRA_EXT’
   EXTRA_EXT(ARB_compute_shader);
   ^~~~~~~~~

Not really sure what we should do about that one.

Rob Clark (10):
  util: fix new gcc6 warnings
  gallium/hud: fix new gcc6 warnings
  gallium/auxiliary: fix new gcc6 warnings
  trace: fix new gcc6 warnings
  postprocess: fix new gcc6 warnings
  freedreno/ir3: fix new gcc6 errors
  i965: fix new gcc6 warnings
  glsl: fix new gcc6 warnings
  glsl: fix new gcc6 warnings
  mesa: fix new gcc6 warnings

 src/compiler/glsl/ast_to_hir.cpp                      |  2 +-
 src/compiler/glsl/lower_discard_flow.cpp              | 12 ------------
 src/gallium/auxiliary/hud/font.c                      |  2 ++
 src/gallium/auxiliary/pipebuffer/pb_bufmgr_mm.c       |  4 ++--
 src/gallium/auxiliary/postprocess/filters.h           |  2 +-
 src/gallium/drivers/freedreno/ir3/ir3_compiler_nir.c  |  1 -
 src/gallium/drivers/trace/tr_context.c                |  7 -------
 src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp |  2 +-
 src/mesa/main/texstore.c                              |  3 ---
 src/util/hash_table.h                                 |  2 +-
 10 files changed, 8 insertions(+), 29 deletions(-)

-- 
2.5.0



More information about the mesa-dev mailing list