[Mesa-dev] [PATCH 00/15] Clang compiler warning fixes

Mike Lothian mike at fireburn.co.uk
Tue Apr 11 23:13:15 UTC 2017


When trying to test these I notice that even without the patches clover
isn't compiling

/var/tmp/portage/media-libs/mesa-9999/work/mesa-9999/src/gallium/state_trackers/clover/llvm/codegen/common.cpp:132:54:
error: ‘Offset’ is not a member of ‘clang::LangAS’
                                                   -
clang::LangAS::Offset]) {
                                                     ^~~~~


On Tue, 11 Apr 2017 at 19:11 Brian Paul <brianp at vmware.com> wrote:

> On 04/11/2017 11:58 AM, Samuel Pitoiset wrote:
> > Noticed while building my whole ARB_bindless_texture work to avoid
> > missing switch cases, etc. This series doesn't fix all warnings, just
> > the easy ones.
> >
> > Please, review!
> > Thanks.
>
> For 1-14, Reviewed-by: Brian Paul <brianp at vmware.com>
>
> -Brian
>
>
> >
> > Samuel Pitoiset (15):
> >    virgl: add missing PIPE_CAP_DOUBLES
> >    mesa: remove unused _mesa_unmarshal_BindBufferBase()
> >    mesa: remove unused clamp_float_to_uint() and clamp_half_to_uint()
> >    mesa: remove some unused functions in the perf monitor area
> >    draw: remove unused overflow()
> >    draw: remove unused wideline_stage()
> >    trace: remove some unused trace_dump_tag*() functions
> >    softpipe: remove unused get_texel_quad_2d()
> >    softpipe: remove unused quad_shade_stage()
> >    softpipe: remove unused sp_exec_fragment_shader()
> >    llvmpipe: remove unused subpixel_snap() and fixed_to_float()
> >    svga: remove unused vmw_dri1_intersect_src_bbox()
> >    radeon: remove duplicate 'const' specifier
> >    st/mesa: fix wrong comparison in update_framebuffer_state()
> >    i965: add missing ir_unop_*/ir_binop_* in visit_leave()
> >
> >   src/gallium/auxiliary/draw/draw_pipe_vbuf.c        |  8 ----
> >   src/gallium/auxiliary/draw/draw_pipe_wide_line.c   |  8 ----
> >   src/gallium/drivers/llvmpipe/lp_setup_tri.c        | 12 -----
> >   src/gallium/drivers/softpipe/sp_fs_exec.c          |  8 ----
> >   src/gallium/drivers/softpipe/sp_quad_fs.c          |  8 ----
> >   src/gallium/drivers/softpipe/sp_tex_sample.c       | 17 -------
> >   src/gallium/drivers/trace/tr_dump.c                | 52
> ----------------------
> >   src/gallium/drivers/virgl/virgl_screen.c           |  1 +
> >   src/gallium/winsys/svga/drm/vmw_screen_dri.c       | 32 -------------
> >   .../dri/i965/brw_fs_channel_expressions.cpp        |  3 ++
> >   .../drivers/dri/radeon/radeon_common_context.c     |  2 +-
> >   .../drivers/dri/radeon/radeon_common_context.h     |  2 +-
> >   src/mesa/main/marshal.c                            |  8 ----
> >   src/mesa/main/pack.c                               | 15 -------
> >   src/mesa/main/performance_monitor.c                | 27 -----------
> >   src/mesa/state_tracker/st_atom_framebuffer.c       |  8 ++--
> >   16 files changed, 10 insertions(+), 201 deletions(-)
> >
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170411/f36a07bf/attachment.html>


More information about the mesa-dev mailing list