Re: [Tinycc-devel] patches for discussion

[Michael Matz]

Hi,

[I'm currently having problems getting mails through to the GNU mail 
server, sorry if you get this multiple times]

On Fri, 13 May 2016, Sergey Korshunoff wrote:

> I have the following patches in github repo (requests in tinycc bugzilla):
>
> + memory model macros __{L,}LP64__
> https://github.com/seyko2/tinycc/commit/eb2a3a8ae941817a5bea7e561999b4c463873a0f

Please push to mob.

> + short_call_convention 
> https://github.com/seyko2/tinycc/commit/558614d3f822de03bb6267f47a24487faa69d29a

This is okay as well.  A bit ugly, and potentially a problem on other 
archs as well.

> + --whole-archive support 
> https://github.com/seyko2/tinycc/commit/65e42df4d0fedb340f42652bb49fdacc7bc3e05c

In principle I have nothing against including this feature, but I don't 
like the actual implementation; it's got too much churn.  Instead add a 
new flag in the TCCState structure, set it when seeing --whole-archive, 
unset it when seeing --no-whole-archive and test it in tcc_load_alacarte. 
The whole patch including docu shouldn't need more than 30 lines and no 
changes in function signatures.

> PS: there is request (Bugzilla) to implement
> bug #37244: tinycc: lacks -nostartfiles -nodefaultlibs support, needed by Wine
> http://savannah.nongnu.org/bugs/?37244

Hmm?  tcc already has -nostdlib disabling automatic inclusion of the crt 
files and the C library.  The bug report is meanwhile bogus.


Ciao,
Michael.