This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PowerPC] Tidy unwind support
- From: Geoffrey Keating <geoffk at geoffk dot org>
- To: Alan Modra <amodra at bigpond dot net dot au>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: 01 Sep 2004 11:48:53 -0700
- Subject: Re: [PowerPC] Tidy unwind support
- References: <20040901050711.GN26692@bubble.modra.org>
Alan Modra <amodra@bigpond.net.au> writes:
> This patch removes one of the annoyances when bootstrapping a
> powerpc*-linux toolchain, the need for glibc headers.
>
> * config/rs6000/linux64.h: Don't include signal.h or sys/ucontext.h.
> (struct kernel_old_ucontext): Delete.
> (struct gcc_pt_regs, gcc_sigcontext, gcc_ucontext): New.
> (MD_FALLBACK_FRAME_STATE_FOR): Use gcc_* structs. Only define
> when IN_LIGGCC2.
>
> Verified by comparing objdump -dr results for old and new unwind-dw2.o,
> all multilibs. I haven't provided the equivalent linux.h patch here,
> because I'd like to move the unwind defines to another file,
> config/rs6000/linux-unwind.h, saving the duplication. Then either add
> the new file to the list of tm_file's in config.gcc, or include into
> unwind-dw2.c via MD_UNWIND_SUPPORT as I did with my ill-thought patch
> in http://gcc.gnu.org/ml/gcc-patches/2004-08/msg02541.html. Either way,
> I'll commit this first to better track the changes.
Have you considered doing what ppc-darwin does, which is to move this
large and complicated macro into a function in its own .c file?