This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch] i386go32_frame_saved_pc()


> Date: Tue, 04 Dec 2001 11:35:43 -0500
> From: Andrew Cagney <ac131313@cygnus.com>
> 
> >>+   return read_memory_integer (frame->frame + 4, 4);
> 
> > 
> > Just one small question, shouldn't it also be an
> > read_memory_unsigned_integer here ?
> > a CORE_ADDR for i386 is always unsigned, no ?
> 
> Quiet possibly, a question for Eli.

It should be unsigned, but it doesn't matter in practice, since the
DJGPP library function used to read memory accepts an unsigned
argument.  So the compiler, seeing its prototype, should generate the
right code, and everything's dandy ;-)


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]