[980307]: Warning patches #1/5 - signed vs. unsigned.

Jim Wilson wilson@cygnus.com
Wed Mar 11 15:15:00 GMT 1998


This is OK to install.

I see that you have an enquire.c change.  The enquire.c copyright requires
that we document all changes near the top of the file.  Please add such a
change similar to the existing changes.

	In dwarf2out.c (reg_save) I added a FIXME comment to highlight a
	comparison of an unsigned variable with -1 in an if statement!

	!   if (sreg == -1)	/* FIXME: How can this be -1? It is an unsigned!  */

	Can someone who's more experienced with this particular code take a look?

The code is correct.  -1 is used to indicate that the value isn't a register
number.  The -1 just gets casted to unsigned, and becomes UINT_MAX.  It may
not be pretty, but I think it should always work.

Jim



More information about the Gcc-bugs mailing list