This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [patch] i386go32_frame_saved_pc()
- From: "Eli Zaretskii" <eliz at is dot elta dot co dot il>
- To: ac131313 at cygnus dot com
- Cc: muller at cerbere dot u-strasbg dot fr, gdb-patches at sources dot redhat dot com
- Date: Tue, 04 Dec 2001 19:46:24 +0200
- Subject: Re: [patch] i386go32_frame_saved_pc()
- References: <4.2.0.58.20011204162131.00acaa68@ics.u-strasbg.fr> <3C0CFB5F.8090401@cygnus.com>
- Reply-to: Eli Zaretskii <eliz at is dot elta dot co dot il>
> 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 ;-)