Re: [PATCH] Add support for .cfi_sections - way to emit also .debug_frame from .cfi_* directives

[Matthias Klose <doko at ubuntu dot com>]

On 01.10.2009 22:58, Jakub Jelinek wrote:
> On Thu, Oct 01, 2009 at 12:21:30PM -0400, Daniel Jacobowitz wrote:
> > > +/* By default emit .eh_frame only, not .eh_frame.  */
> > > +static int cfi_sections = 1;
> >
> > Typo.
>
> Thanks, fixed.
>
> > Don't suppose I could persuade you to give these names instead of
> > numbers?  ".cfi_sections .eh_frame" and a hypothetical
> > ".cfi_sections .debug_frame,.ARM.extab"?
>
> Like this?
>
> 2009-10-01  Jakub Jelinek<jakub@redhat.com>
>
> 	* dw2gencfi.c: Include dwarf2dbg.h.
> 	(DWARF2_FORMAT): Define if not defined.
> 	(dot_cfi_sections): New function.
> 	(cfi_pseudo_table): Handle .cfi_sections.
> 	(CFI_EMIT_eh_frame, CFI_EMIT_debug_frame): Define.
> 	(cfi_sections): New variable.
> 	(output_cie, output_fde, select_cie_for_fde): Add eh_frame
> 	argument, add supporting for outputting .debug_frame
> 	section.
> 	(cfi_change_reg_numbers): New function or macro.
> 	(cfi_finish): Only emit .eh_frame if
> 	cfi_sections&  CFI_EMIT_eh_frame.  Emit .debug_frame if
> 	cfi_sections&  CFI_EMIT_debug_frame.
> 	* config/tc-ppc.h (md_reg_eh_frame_to_debug_frame): Define.
> 	* doc/as.texinfo (CFI directives): Document .cfi_sections.

this fails to build on powerpc-linux:

../../gas/dw2gencfi.c: In function 'cfi_finish':
../../gas/dw2gencfi.c:1721: error: 'struct fde_entry' has no member named 'first'
make[5]: *** [dw2gencfi.o] Error 1