[Top][All Lists]

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

Re: [Tinycc-devel] Annoying new warning

From: Christian Jullien
Subject: Re: [Tinycc-devel] Annoying new warning
Date: Tue, 2 May 2017 14:24:22 +0200

When I see this warning, it makes me think that application will produce
unpredictable results based on value on stack.
That's why I consider this as an annoying warning.
It's hard to figure out that this code is platform dependent and will not
happen with mine (ARM).
Suppose that tomorrow, someone else will use this function for all targets,
warning will remain the same but will produce wrong code.

IMHO it must be fixed before next release.

Note this warning is rather recent, like few weeks ago.

-----Original Message-----
From: Tinycc-devel [mailto:address@hidden
On Behalf Of grischka
Sent: mardi 2 mai 2017 14:05
To: address@hidden
Subject: Re: [Tinycc-devel] Annoying new warning

Christian Jullien wrote:
> So do you agree if I push this patch which works in all cases (even 
> with dead code :o)

> +    dl1.l.upper = 0;
> +    dl1.l.lower = 0;

No, please, we do not fix warnings in functions for platforms where the
function cannot be used as pointed out just by that warning.

What do you think should other people think if they see this?

--- grischka

Tinycc-devel mailing list

reply via email to

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