On 02/10/2006, at 3:37 PM, Jack Howarth wrote:

Geoff,
     I made one typo in my original proposed patch for
unwind-dw2-fde-darwin.c. It should be...

Index: unwind-dw2-fde-darwin.c
===================================================================
--- unwind-dw2-fde-darwin.c     (revision 117350)
+++ unwind-dw2-fde-darwin.c     (working copy)
@@ -61,7 +61,7 @@ struct mach_header;
 struct mach_header_64;
 extern char *getsectdatafromheader (struct mach_header*, const char*,
                        const char *, unsigned long *);
-extern char *getsectdatafromheader_64 (struct mach_header*, const char*, +extern char *getsectdatafromheader_64 (struct mach_header_64*, const char*,
                        const char *, unsigned long *);

 /* This is referenced from KEYMGR_GCC3_DW2_OBJ_LIST.  */
@@ -157,8 +157,12 @@ examine_objects (void *pc, struct dwarf_

/* For ppc only check whether or not we have __DATA eh frames. */
 #ifdef __ppc__
+#if __LP64__
+ fde = getsectdatafromheader_64 ((struct mach_header_64 *)image->mh, "__DATA", "__eh_frame", &sz);
+#else
fde = getsectdatafromheader (image->mh, "__DATA", "__eh_frame", &sz);
 #endif
+#endif

        if (fde == NULL)
          {

My reply to the original message seems to have gotten lost.

What I was trying to say is that __ppc__ on Darwin means only 32-bit powerpc. It should not be defined for 64-bit powerpc, that's __ppc64__.

Attachment: smime.p7s
Description: S/MIME cryptographic signature

Reply via email to