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:tinycc-devel-bounces+eligis=orange...@nongnu.org] On Behalf Of grischka Sent: mardi 2 mai 2017 14:05 To: tinycc-devel@nongnu.org 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 Tinycc-devel@nongnu.org https://lists.nongnu.org/mailman/listinfo/tinycc-devel _______________________________________________ Tinycc-devel mailing list Tinycc-devel@nongnu.org https://lists.nongnu.org/mailman/listinfo/tinycc-devel
