[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Tinycc-devel] TinyCC for bootstrapping (with patches)
From: |
Giovanni Mascellani |
Subject: |
Re: [Tinycc-devel] TinyCC for bootstrapping (with patches) |
Date: |
Mon, 14 Jan 2019 11:09:25 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.4.0 |
Hi,
Il 13/01/19 19:37, Daniel Glöckner ha scritto:
> I really like the idea of using a library to perform floating point
> operations and I also like that you made it optional.
Good!
> But it looks like you made the assumption that long double is an
> 80 bit floating point type. This is only true for x86-32. You have
> to look at the LDOUBLE_SIZE macro to determine which of float64_t,
> extFloat80_t, and float128_t to use (For x86-32 it is 12 instead of
> 10 because of padding).
I agree. As I said in my first email, for the moment the patch is
Intel-specific (and actually i386-specific). I was asking for
preliminary opinions, because if people immediately think that such a
patch would never be accepted, then I would not bother generalizing it.
BTW, it is correct to assume that for all architecture supported by tcc
float and double are always 32 and 64 bits?
Thanks, Giovanni.
--
Giovanni Mascellani <address@hidden>
Postdoc researcher - Université Libre de Bruxelles
signature.asc
Description: OpenPGP digital signature