This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [RFA/pa-risc] Correct incorrect frame base
- From: Daniel Jacobowitz <drow at false dot org>
- To: Joel Brobecker <brobecker at adacore dot com>
- Cc: gdb-patches at sources dot redhat dot com
- Date: Fri, 31 Dec 2004 11:12:58 -0500
- Subject: Re: [RFA/pa-risc] Correct incorrect frame base
- References: <20041231151415.GL1824@adacore.com>
On Fri, Dec 31, 2004 at 07:14:15PM +0400, Joel Brobecker wrote:
> There is a small confusion in the source register for the addil
> instruction. The actual sequence should be
>
> addil high21,%r30
> ldo low11(%r1),%r30
>
> I think the confusion comes from the fact that the result of
> the sum is stored in %r1.
"addil <number>,%r30" sets %r1 and not %r30? Whose bright idea was
that, anyway?
> Anyway, I replaced r1 by r30 in the check above, and that fixed
> the problem.
>
> 2004-12-31 Joel Brobecker <brobecker@gnat.com>
>
> * hppa-tdep.c (prologue_inst_adjust_sp): Fix small confusion
> in register number for ldil instruction.
>
> Tested on HP/UX 11.00, no regression.
> OK to apply?
OK. Typo in the changelog: that's addil, not ldil, isn't it?
--
Daniel Jacobowitz