Re: [Tinycc-devel] tiny bit of lint

2017-05-07 Thread Christian Jullien
Hi Larry, I share this hobby with you but only with my own projects. If you like to chase warnings in C, I invite you to forget those 'chicken' checks and to move to something stronger such as (from easier to stronger): * clang and --analyze * VC++ -analyse * smatch that helped to find ~3000

Re: [Tinycc-devel] tiny bit of lint

2017-05-07 Thread Michael Matz
Hi, On Sun, 7 May 2017, Larry Doolittle wrote: Using gcc's -Wextra, I found 17 "unused parameter", which could be addressed by bloating the source code a bit (casting to void), and with no consequence on the binary. If anyone else thought that would be an improvement, I'd be happy to write

[Tinycc-devel] ll-bitfield test on i386-linux

2017-05-07 Thread Michael Matz
Hello grischka, parts of your last patch: * tcctest.c: remove outdated limitation for ll-bitfield test It always worked, there is no reason why it should not work in future. That's not true. The comment I've put there tried to explain it: -#ifndef __i386__ -/* on i386 we

Re: [Tinycc-devel] Add support of musl-libc to tinycc

2017-05-07 Thread Larry Doolittle
Michael - On Sun, May 07, 2017 at 08:16:08AM +0200, Michael Matz wrote: > On debian, if you have libc6-dev and musl-dev installed you get a > [mess that doesn't work with tinycc]. > So, yeah, the detection/support of musl > is early and doesn't work with the split-mind approach that debian > is

Re: [Tinycc-devel] Annoying new warning

2017-05-07 Thread Christian Jullien
Ok, it works, including tests with: jullien@sims3:~/tinycc $ git diff diff --git a/arm-gen.c b/arm-gen.c index a5dabd1..34e1ef2 100644 --- a/arm-gen.c +++ b/arm-gen.c @@ -2150,6 +2150,4 @@ ST_FUNC void gen_vla_alloc(CType *type, int align) { #endif

Re: [Tinycc-devel] Annoying new warning

2017-05-07 Thread grischka
Christian Jullien wrote: alloca-arm.S: error: unrecognized file type Ok, bottom of arm-gen.c, remove #ifndef TCC_IS_NATIVE thanks. Makefile:67: recipe for target 'alloca-arm.o' failed -Original Message- From: Tinycc-devel

Re: [Tinycc-devel] Annoying new warning

2017-05-07 Thread Christian Jullien
Thank grischka, Be use I annoyed you with this warning because, to me, it exhibits a potential issue on arm. Your fix gives: make[1]: Entering directory '/home/jullien/tinycc/lib' ../tcc -c libtcc1.c -o libtcc1.o -DCONFIG_TRIPLET="arm-linux-gnueabihf" -DTCC_TARGET_ARM -DTCC_ARM_EABI

Re: [Tinycc-devel] Annoying new warning

2017-05-07 Thread grischka
Christian Jullien wrote: So, gentlemen, it seems that at least one ARM configuration/option requires the offending code having test on an uninitialized value. Commenting using #if will fail if "arm-fpa-ld" is used. Can you please take a moment to investigate and propose the right fix. Nope.

Re: [Tinycc-devel] Annoying new warning

2017-05-07 Thread Christian Jullien
So, gentlemen, it seems that at least one ARM configuration/option requires the offending code having test on an uninitialized value. Commenting using #if will fail if "arm-fpa-ld" is used. Can you please take a moment to investigate and propose the right fix. C. -Original Message-

Re: [Tinycc-devel] Annoying new warning

2017-05-07 Thread grischka
Christian Jullien wrote: Especially if " LDOUBLE_SIZE != 8 is not possible in any arm config" why is there such a test in arm-gen.c code? The "arm-fpa-ld" variant gets LDOUBLE_SIZE=12 passed from the Makefile. -- gr ___ Tinycc-devel mailing list

Re: [Tinycc-devel] Annoying new warning

2017-05-07 Thread Christian Jullien
Michael, Thank you for patience with this thread. Your previous reply was very clear about my mistake. I just wanted to point out the inconsistency of arm code. Especially if " LDOUBLE_SIZE != 8 is not possible in any arm config" why is there such a test in arm-gen.c code? I also would prefer a

Re: [Tinycc-devel] Annoying new warning

2017-05-07 Thread Michael Matz
Hi, On Sun, 7 May 2017, Christian Jullien wrote: Forget one moment what has been said so far and concentrate on my concern with narrowed info. Facts: Yes, all known and agreed upon. * Finally, arm-gen.c uses TOK___fixxfdi with yet another #if condition => LDOUBLE_SIZE != 8 #if

Re: [Tinycc-devel] Add support of musl-libc to tinycc

2017-05-07 Thread Michael Matz
Hi, On Sat, 6 May 2017, Larry Doolittle wrote: Micheal, Marc, and tinycc lurkers - On Sun, May 07, 2017 at 04:54:38AM +0200, Michael Matz wrote: On Thu, 20 Apr 2017, Marc Vertes wrote: I just committed the patch to support musl-libc in the tcc compiler development branch: