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: [RFA/pa-risc] Correct incorrect frame base


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


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