Paolo Bonzini <pbonz...@redhat.com> writes:
> On 31/01/2017 12:40, Markus Armbruster wrote:
>>>
>>> #define QEMU_NORETURN __attribute__ ((__noreturn__))
>>>
>>> -#if QEMU_GNUC_PREREQ(3, 4)
>>> #define QEMU_WARN_UNUSED_RESULT __attribute__((warn_unused_result))
>>> -#else
>>> -#define QEMU_WARN_UNUSED_RESULT
>>> -#endif
>> Should we inline this macro?
>>
>>>
>>> -#if QEMU_GNUC_PREREQ(4, 0)
>>> #define QEMU_SENTINEL __attribute__((sentinel))
>>> -#else
>>> -#define QEMU_SENTINEL
>>> -#endif
>> Likewise.
>
> Why, since we don't do that for QEMU_NORETURN, QEMU_PACKED, etc.?
Because we do it for aligned, always_inline, constructor, format, mode,
noinline, and in places even noreturn and packed:
$ git-grep __attribute__ | sed '/define/d;s/.*__attribute__
*((\([A-Za-z0-9_]*\).*/\1/' | sort -u
