All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paolo Bonzini <pbonzini@redhat.com>
To: John Snow <jsnow@redhat.com>,
	Peter Maydell <peter.maydell@linaro.org>,
	qemu-devel <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] [PULL v3 00/33] Misc patches for 2016-02-08
Date: Wed, 10 Feb 2016 22:33:16 +0100	[thread overview]
Message-ID: <56BBAC9C.9080908@redhat.com> (raw)
In-Reply-To: <56BB8C8F.8000803@redhat.com>



On 10/02/2016 20:16, John Snow wrote:
> 
> jsnow@scv ((977a82a...)) ~/s/q/b/git> ../../configure --cxx=clang++
> --cc=clang --host-cc=clang --extra-cflags=-Werror
> --extra-cflags=-fsanitize=undefined
> --extra-cflags=-Wno-deprecated-declarations
> --extra-cflags=-fno-sanitize=float-divide-by-zero
> --target-list="x86_64-softmmu"
> 
> ERROR: sizeof(size_t) doesn't match GLIB_SIZEOF_SIZE_T.
>        You probably need to set PKG_CONFIG_LIBDIR
>        to point to the right pkg-config files for your
>        build target
> 
> 
> 
> Problem appears to be this:
> 
> /usr/include/glib-2.0/glib/gmem.h:76:58: error: unknown attribute
> '__alloc_size__' ignored [-Werror,-Wunknown-attributes]
> gpointer g_malloc         (gsize         n_bytes) G_GNUC_MALLOC
> G_GNUC_ALLOC_SIZE(1);
>                                                                 ^
> /usr/include/glib-2.0/glib/gmacros.h:67:45: note: expanded from macro
> 'G_GNUC_ALLOC_SIZE'
> #define G_GNUC_ALLOC_SIZE(x) __attribute__((__alloc_size__(x)))

F23 wraps it like this:

/* Clang feature detection: http://clang.llvm.org/docs/LanguageExtensions.html */
#ifndef __has_feature
#define __has_feature(x) 0
#endif

#if     (!defined(__clang__) && ((__GNUC__ > 4) || (__GNUC__ == 4 && __GNUC_MINO
R__ >= 3))) || \
        (defined(__clang__) && __has_feature(__alloc_size__))
#define G_GNUC_ALLOC_SIZE(x) __attribute__((__alloc_size__(x)))
#define G_GNUC_ALLOC_SIZE2(x,y) __attribute__((__alloc_size__(x,y)))
#else
#define G_GNUC_ALLOC_SIZE(x)
#define G_GNUC_ALLOC_SIZE2(x,y)
#endif

Please open a bug on the distro to backport this fix.

Paolo

      reply	other threads:[~2016-02-10 21:33 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-09 16:13 [Qemu-devel] [PULL v3 00/33] Misc patches for 2016-02-08 Paolo Bonzini
2016-02-09 16:13 ` [Qemu-devel] [PULL 20/33] target-i386: Access segs via TCG registers Paolo Bonzini
2016-02-09 16:13 ` [Qemu-devel] [PULL 31/33] get_maintainer.pl: fall back to git if only lists are found Paolo Bonzini
2016-02-09 16:13 ` [Qemu-devel] [PULL 32/33] MAINTAINERS: add all-match entry for qemu-devel@ Paolo Bonzini
2016-02-10 12:48 ` [Qemu-devel] [PULL v3 00/33] Misc patches for 2016-02-08 Paolo Bonzini
2016-02-10 13:49   ` Peter Maydell
2016-02-10 14:10     ` Paolo Bonzini
2016-02-10 19:16   ` John Snow
2016-02-10 21:33     ` Paolo Bonzini [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=56BBAC9C.9080908@redhat.com \
    --to=pbonzini@redhat.com \
    --cc=jsnow@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.