[PATCH] Fix frame ID comparison problem on s390

Daniel Jacobowitz drow@false.org
Sun Jun 27 21:48:00 GMT 2004


On Sun, Jun 27, 2004 at 10:47:58PM +0200, Ulrich Weigand wrote:
> Andrew Cagney wrote:
> 
> > with those changes its ok,
> 
> Thanks; here's the version I finally committed.
> 
> (Sorry it took so long, I got sidetracked by a kernel bug triggered
> by running the gdb test suite that caused random kernel memory 
> corruption :-/)

Thanks!  Just one small nit.  If you're going to do this:

> +   id.stack_addr_p = 1;
>     id.code_addr = code_addr;
> +   id.code_addr_p = 1;
>     id.special_addr = special_addr;
> +   id.special_addr_p = 1;

Then these should be "unsigned int : 1":

> +   /* Flags to indicate the above fields have valid contents.  */
> +   int stack_addr_p : 1;
> +   int code_addr_p : 1;
> +   int special_addr_p : 1;
>   };
>   
>   /* Methods for constructing and comparing Frame IDs.

Some future version of GCC will warn about this mistake.  The only
valid values for "int : 1" are 0 and -1.

-- 
Daniel Jacobowitz



More information about the Gdb-patches mailing list