[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute
From: |
M. Warner Losh |
Subject: |
Re: [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute |
Date: |
Sun, 30 Nov 2008 10:21:59 -0700 (MST) |
In message: <address@hidden>
Stefan Weil <address@hidden> writes:
: Jan Kiszka schrieb:
: > Laurent Desnogues wrote:
: >> On Sun, Nov 30, 2008 at 12:51 PM, Stefan Weil <address@hidden>
: >> wrote:
: >>> Why do we need a new macro instead of just using __attribute__
: >>> ((__noreturn__))?
: >> Don't we need it to prevent gcc-ism from being all over the code?
: >
: > Yep, and that's also why we need wrapping. You can easily define it away
: > if your compiler doesn't support it. We just need to add the required
: > conditions.
: >
: >> Though I wonder how much tcg converted code still depends on
: >> being compiled by gcc.
: >
: > I guess once the hard dependencies are removed, fixing things like the
: > existing __attributes__ etc. will just be mechanical work.
: >
: > Jan
: >
:
:
: For compilers which don't support __attributes__, a simple
:
: #define __attributes__(dummy) /* dummy */
:
: or an equivalent command line option would eliminate all gcc-isms.
: I don't think we need wrapping, at least not now.
This doesn't work so well when 'dummy' is __packed__.
The BSD projects have had similar macros for a long long time, and
they work out well in practice.
Warner
- [Qemu-devel] [PATCH 2/2] Add __noreturn function attribute, Jan Kiszka, 2008/11/28
- Re: [Qemu-devel] [PATCH 2/2] Add __noreturn function attribute, malc, 2008/11/28
- [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute, Jan Kiszka, 2008/11/28
- Re: [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute, Thiemo Seufer, 2008/11/28
- Re: [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute, Avi Kivity, 2008/11/30
- Re: [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute, Stefan Weil, 2008/11/30
- Re: [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute, Laurent Desnogues, 2008/11/30
- [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute, Jan Kiszka, 2008/11/30
- Re: [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute, Stefan Weil, 2008/11/30
- Re: [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute, Andreas Färber, 2008/11/30
- Re: [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute,
M. Warner Losh <=
- [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute, Jan Kiszka, 2008/11/30
- Re: [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute, Thiemo Seufer, 2008/11/30
- Re: [Qemu-devel] Re: [PATCH 2/2] Add __noreturn function attribute, Jan Kiszka, 2008/11/30