Re: [Tinycc-devel] Annoying new warning

[Christian Jullien]

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
address@hidden
https://lists.nongnu.org/mailman/listinfo/tinycc-devel