For several files in sys/dev/drm, disable -Wunused-value when building

with clang.  There are several macros in these files that return values,
and in some cases nothing is done with them, but it is completely
harmless.  For some other files, also disable -Wconstant-conversion,
since that triggers a false positive with the DMA_BIT_MASK() macro.

MFC after:	1 week
This commit is contained in:
Dimitry Andric 2011-12-30 01:54:45 +00:00
parent 092c20cb19
commit 874fb554f7
5 changed files with 20 additions and 4 deletions

View File

@ -992,15 +992,18 @@ dev/drm/mga_irq.c optional mgadrm
dev/drm/mga_state.c optional mgadrm \
compile-with "${NORMAL_C} -finline-limit=13500"
dev/drm/mga_warp.c optional mgadrm
dev/drm/r128_cce.c optional r128drm
dev/drm/r128_cce.c optional r128drm \
compile-with "${NORMAL_C} ${NO_WUNUSED_VALUE} ${NO_WCONSTANT_CONVERSION}"
dev/drm/r128_drv.c optional r128drm
dev/drm/r128_irq.c optional r128drm
dev/drm/r128_state.c optional r128drm \
compile-with "${NORMAL_C} -finline-limit=13500"
compile-with "${NORMAL_C} ${NO_WUNUSED_VALUE} -finline-limit=13500"
dev/drm/r300_cmdbuf.c optional radeondrm
dev/drm/r600_blit.c optional radeondrm
dev/drm/r600_cp.c optional radeondrm
dev/drm/radeon_cp.c optional radeondrm
dev/drm/r600_cp.c optional radeondrm \
compile-with "${NORMAL_C} ${NO_WUNUSED_VALUE} ${NO_WCONSTANT_CONVERSION}"
dev/drm/radeon_cp.c optional radeondrm \
compile-with "${NORMAL_C} ${NO_WUNUSED_VALUE} ${NO_WCONSTANT_CONVERSION}"
dev/drm/radeon_cs.c optional radeondrm
dev/drm/radeon_drv.c optional radeondrm
dev/drm/radeon_irq.c optional radeondrm

View File

@ -20,6 +20,7 @@ NO_WCONSTANT_CONVERSION= -Wno-constant-conversion
NO_WARRAY_BOUNDS= -Wno-array-bounds
NO_WSHIFT_COUNT_NEGATIVE= -Wno-shift-count-negative
NO_WSHIFT_COUNT_OVERFLOW= -Wno-shift-count-overflow
NO_WUNUSED_VALUE= -Wno-unused-value
# Several other warnings which might be useful in some cases, but not severe
# enough to error out the whole kernel build. Display them anyway, so there is
# some incentive to fix them eventually.

View File

@ -6,3 +6,7 @@ SRCS = r128_cce.c r128_drv.c r128_irq.c r128_state.c
SRCS +=device_if.h bus_if.h pci_if.h opt_drm.h
.include <bsd.kmod.mk>
CWARNFLAGS.r128_cce.c= ${NO_WUNUSED_VALUE} ${NO_WCONSTANT_CONVERSION}
CWARNFLAGS.r128_state.c= ${NO_WUNUSED_VALUE}
CWARNFLAGS+= ${CWARNFLAGS.${.IMPSRC:T}}

View File

@ -7,3 +7,7 @@ SRCS = r300_cmdbuf.c r600_blit.c r600_cp.c radeon_cp.c radeon_cs.c \
SRCS +=device_if.h bus_if.h pci_if.h opt_drm.h
.include <bsd.kmod.mk>
CWARNFLAGS.r600_cp.c= ${NO_WUNUSED_VALUE} ${NO_WCONSTANT_CONVERSION}
CWARNFLAGS.radeon_cp.c= ${NO_WUNUSED_VALUE} ${NO_WCONSTANT_CONVERSION}
CWARNFLAGS+= ${CWARNFLAGS.${.IMPSRC:T}}

View File

@ -20,3 +20,7 @@ opt_drm.h:
echo $(DRM_LINUX_OPT) >> opt_drm.h
.include <bsd.kmod.mk>
CWARNFLAGS.via_dma.c= ${NO_WUNUSED_VALUE}
CWARNFLAGS.via_dmablit.c= ${NO_WUNUSED_VALUE}
CWARNFLAGS+= ${CWARNFLAGS.${.IMPSRC:T}}