[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Tinycc-devel] Using tinycc for full source bootstrapping

From: Larry Doolittle
Subject: Re: [Tinycc-devel] Using tinycc for full source bootstrapping
Date: Mon, 25 Sep 2017 22:25:27 -0700
User-agent: Mutt/1.5.21 (2010-09-15)

On Sun, Sep 24, 2017 at 07:40:32PM +0200, grischka wrote:
> Jan Nieuwenhuizen wrote:
> >Are you willing to consider discussing/taking patches that reduce the
> >variety and complexity of C constructs used in TinyCC source code?
> >I have a list of about 20 unpolished patches[0] for constructs like:
> I wonder which would be the smaller patch:  The changes you propose
> for tcc or a patch to support these constructs as is in the bootstrap
> compiler.

I wonder that too.  I looked through a lot of the changes you made,
and 99% of them decrease the code quality of tcc.  IMHO the long-term
path forward would be to improve your bootstrap compiler.

The comma operator example you posted, that confused me at first,
is an exception, and I just pushed a patch.  Using semicolons here
is more of a "say what you mean" and "principle of least surprise"
approach to coding this function.  The _other_ comma operators you found
(and eliminated) in tcc's code base are used for all the right reasons,
to reduce code duplication and create a smooth flow of control.

I enthusiastically support boostrap projects in general, and wish you
the best of luck going forward!  Just don't expect mainline tcc to
regress its code quality for your benefit.

  - Larry

reply via email to

[Prev in Thread] Current Thread [Next in Thread]