This is the mail archive of the binutils@sourceware.org mailing list for the binutils 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]

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


On Thu, Oct 01, 2009 at 05:38:21PM +0200, Jakub Jelinek wrote:
> Hi!
> 
> This patch attempts to address http://gcc.gnu.org/PR40521.

Thanks for working on this!

> +/* By default emit .eh_frame only, not .eh_frame.  */
> +static int cfi_sections = 1;

Typo.

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"?

[The CFI directives are either sufficient, or very nearly so, for the
ARM unwind tables - as long as you don't try to do certain fancy
things with them.  At least one other toolchain uses the same
directives for the tables and for .debug_frame.]

-- 
Daniel Jacobowitz
CodeSourcery


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]