This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [RFA] ARM : prologue scan
- From: Jerome Guitton <guitton at act-europe dot fr>
- To: Richard dot Earnshaw at arm dot com
- Cc: gdb-patches at sources dot redhat dot com
- Date: Tue, 9 Sep 2003 12:23:15 +0200
- Subject: Re: [RFA] ARM : prologue scan
- References: <20030722114709.GB3100@act-europe.fr> <200309051014.h85AEGl21565@pc960.cambridge.arm.com>
Richard Earnshaw (rearnsha@arm.com):
> > 2003-07-21 J. Guitton <guitton@gnat.com>
> >
> > * arm-tdep.c (arm_skip_prologue): Add the handling of "sub ip, sp #n"
> > and "add ip, sp #n", as these instructions can be found in a ATPCS
> > compliant prologue.
> > (arm_scan_prologue): Ditto.
>
> Secondly, and this applies only to the ChangeLog entry itself, this entry
> sequence is nothing to do with the ATPCS (the A*T*PCS doesn't even
> sanction the use of a frame pointer).
You are right. I got confused by these comments:
The APCS (ARM Procedure Call Standard) defines the following
prologue:
mov ip, sp
[stmfd sp!, {a1,a2,a3,a4}]
stmfd sp!, {...,fp,ip,lr,pc}
[stfe f7, [sp, #-12]!]
[stfe f6, [sp, #-12]!]
[stfe f5, [sp, #-12]!]
[stfe f4, [sp, #-12]!]
sub fp, ip, #nn @@ nn == 20 or 4 depending on second insn */
I didn't see this definition in the ARM Thumb Procedure Call Standard...
Is the ARM Procedure Call Standard a different document? If so, what is
its status (Does the ATPCS make the APCS obsolete?) and where can I find
it?
> Anyway, it's not normal
> to put the reason for a change in a CL entry, so just truncate the
> sentence to read:
I used to thought that it was the most important part of the CL entry.
The change itself in the code, the reason in the CL... I guess I am wrong.
> With that change, this is OK.
Thank you very much for your review!
--
Jerome