Re: [Chicken-hackers] [PATCH] Use noreturn attribute in clang versions which support it

2013-10-28 Thread Mario Domenech Goulart
On Thu, 24 Oct 2013 15:14:03 +0200 Peter Bex wrote: > On Wed, Oct 23, 2013 at 11:58:34AM +0200, Peter Bex wrote: >> On Wed, Oct 23, 2013 at 11:53:13AM +0200, Moritz Heidkamp wrote: >> > BTW, I got another warning when running the tests: >> > >> > ../chicken 'embedded4.scm' -output-file 'a.c' -

Re: [Chicken-hackers] [PATCH] Use noreturn attribute in clang versions which support it

2013-10-24 Thread Peter Bex
On Wed, Oct 23, 2013 at 11:58:34AM +0200, Peter Bex wrote: > On Wed, Oct 23, 2013 at 11:53:13AM +0200, Moritz Heidkamp wrote: > > BTW, I got another warning when running the tests: > > > > ../chicken 'embedded4.scm' -output-file 'a.c' -scrutinize -verbose > > -include-path .. > > 'clang' 'a.c

Re: [Chicken-hackers] [PATCH] Use noreturn attribute in clang versions which support it

2013-10-23 Thread Jim Ursetto
On Oct 21, 2013, at 15:27, Peter Bex wrote: > > Hi all, > > Here's another small patch which will properly enable C_noret for clang. > Currently it expands to nothing and causes a slew of warnings mostly > because it doesn't understand that barf() won't return when it detects > an error, and th

Re: [Chicken-hackers] [PATCH] Use noreturn attribute in clang versions which support it

2013-10-23 Thread Peter Bex
On Wed, Oct 23, 2013 at 11:53:13AM +0200, Moritz Heidkamp wrote: > BTW, I got another warning when running the tests: > > ../chicken 'embedded4.scm' -output-file 'a.c' -scrutinize -verbose > -include-path .. > 'clang' 'a.c' -o 'a.o' -c -fno-strict-aliasing -fwrapv > -DHAVE_CHICKEN_CONFIG_H

Re: [Chicken-hackers] [PATCH] Use noreturn attribute in clang versions which support it

2013-10-23 Thread Moritz Heidkamp
Hi, Peter Bex writes: > Fixing this got rid of all the warnings, except one, which exposed > a small bug which is also a minor inefficiency: C_2_divide will call > C_fix() on an integer value, which in case of flonums will not be > initialised. In cases of flonums, it will be initialised, but it

[Chicken-hackers] [PATCH] Use noreturn attribute in clang versions which support it

2013-10-21 Thread Peter Bex
Hi all, Here's another small patch which will properly enable C_noret for clang. Currently it expands to nothing and causes a slew of warnings mostly because it doesn't understand that barf() won't return when it detects an error, and that will leave many variables uninitialised. Luckily, clang h