On Sat, Jan 30, 2016 at 11:29:43PM +1100, David Gibson wrote: > On Fri, Jan 29, 2016 at 02:48:23PM +0000, Peter Maydell wrote: > > On 29 January 2016 at 05:06, David Gibson <da...@gibson.dropbear.id.au> > > wrote: > > > The following changes since commit > > > 357e81c7e880f868833edf9f53cce1f3b09ea8ec: > > > > > > Merge remote-tracking branch 'remotes/cohuck/tags/s390x-20160128' into > > > staging (2016-01-28 11:46:34 +0000) > > > > > > are available in the git repository at: > > > > > > git://github.com/dgibson/qemu.git tags/ppc-for-2.6-20160129 > > > > > > for you to fetch changes up to 1699679e699276c0538008f6ca74cd04e6c68b42: > > > > > > target-ppc: Make every FPSCR_ macro have a corresponding FP_ macro > > > (2016-01-29 14:01:52 +1100) > > > > > > ---------------------------------------------------------------- > > > ppc patch queue for 2016-01-29 > > > > > > Currently accumulated patches for target-ppc, pseries machine type and > > > related devices. > > > * Cleanup of error handling code in spapr > > > * A number of fixes for Macintosh devices for the benefit of MacOS 9 > > > and X > > > * Remove some abuses of the RTAS memory access functions in spapr > > > * Fixes for the gdbstub (and monitor debug) for VMX and VSX extensions. > > > * Fix pseries machine hotplug memory under TCG > > > * Clean up and extend handling of multiple page sizes with 64-bit hash > > > MMUs > > > > > > > Hi. Unfortunately this generates errors when built with clang: > > > > /home/petmay01/linaro/qemu-for-merges/target-ppc/mmu_helper.c:660:20: > > error: unused function 'ppc4xx_tlb_invalidate_virt' > > [-Werror,-Wunused-function] > > static inline void ppc4xx_tlb_invalidate_virt(CPUPPCState *env, > > ^ > > 1 error generated. > > > > The function does appear from a quick grep to be entirely unused... > > > > (GCC doesn't complain about this because it doesn't warn about unused > > static inline functions in a .c file, but clang does.) > > Dammit. Sorry. > > Now.. why didn't travis pick that up :/.
Turns out the answer is because the test for support of #pragma GCC diagnostic turns off -Werror on clang builds for me. So I wonder what's different about your setup that -Werror is working with clang. -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature