[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Chicken-hackers] [PATCH] chicken.h: define C_noret as __attribute__
|
From: |
Mario Domenech Goulart |
|
Subject: |
Re: [Chicken-hackers] [PATCH] chicken.h: define C_noret as __attribute__ ((noreturn)) for clang too |
|
Date: |
Fri, 07 Sep 2012 13:29:04 -0400 |
|
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.1.91 (gnu/linux) |
On Fri, 7 Sep 2012 19:30:03 +0200 Peter Bex <address@hidden> wrote:
> On Thu, Sep 06, 2012 at 09:18:38PM -0400, Mario Domenech Goulart wrote:
>>
>> Christian Lask reported on #chicken that make check fails when Chicken
>> is built with clang (see #917).
>>
>> The attached patch seems to fix that problem (tested with clang 3.0).
>
> I wonder if this will work with older versions of Clang
> (notably XCode's). I'm assuming this #ifdef wasn't added just for
> the hell of it.
Yeah, that's my concern too.
>> In the end, it just silents some warnings that break tests that rely on
>> diff.
>>
>> Even if it is the proper fix, feel free to postpone it to after 4.8.0.
>
> Perhaps the best thing to do.
I agree.
Mario
--
http://parenteses.org/mario