This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc 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] |
Hi On 16 January 2014 20:31, Tom Tromey <tromey@redhat.com> wrote: > svc 0x0 > + cfi_endproc The cfi_endproc placement looks sensible to me. > + cfi_startproc > + cfi_undefined lr The cfi_undefined is missing () and the convenience name 'lr' is not known to GAS, therefore this line should be: cfi_undefined (x30) In the ticket Tom points out that the implementation does not honour the AAPCS, the fix is to insert: mov x29, 0 > #ifdef RESET_PID > tbnz x5, #CLONE_THREAD_BIT, 3f > mov x0, #-1 > @@ -93,7 +96,9 @@ ENTRY(__clone) > > /* We are done, pass the return value through x0. */ > b HIDDEN_JUMPTARGET(_exit) > + cfi_endproc > > + cfi_startproc This part looks fine. Attached is a revised version of your patch including the two changes listed above. If there are no further comments in the next 24hrs or so I'll commit this and address Joseph's comment w.r.t ChangeLog and NEWS. Cheers /Marcus
Attachment:
clone-cfi.txt
Description: Text document
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |