This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH] xtensa: add separate property sections option
- From: Max Filippov <jcmvbkbc at gmail dot com>
- To: "augustine dot sterling at gmail dot com" <augustine dot sterling at gmail dot com>
- Cc: binutils at sourceware dot org, Eric Tsai <erictsai at cadence dot com>, linux-xtensa at linux-xtensa dot org
- Date: Mon, 4 Jun 2018 10:41:06 -0700
- Subject: Re: [PATCH] xtensa: add separate property sections option
- References: <20180530183000.8400-1-jcmvbkbc@gmail.com> <CAGSvup-vArA4oVLK8Ld4AztmFS0TvNgW366=wzOg+Vh3p_x92w@mail.gmail.com>
On Mon, Jun 4, 2018 at 10:19 AM, augustine.sterling@gmail.com
<augustine.sterling@gmail.com> wrote:
> On Wed, May 30, 2018 at 11:30 AM, Max Filippov <jcmvbkbc@gmail.com> wrote:
>>
>> It is currently not possible to correctly match .xt.prop information
>> for sections with identical VMA. Allow creation of separate property
>> sections in the BFD. Add assembler option --separate-prop-tables to
>> allow creation of separate property sections.
>>
>> 2018-05-22 Volodymyr Arbatov <arbatov@cadence.com>
>> bfd/
>>
>> * elf32-xtensa.c (elf32xtensa_separate_props): New global
>> variable.
>> (xtensa_add_names): New function.
>> (xtensa_property_section_name): Add new parameter
>> separate_sections, use it to choose property section name.
>> (xtensa_get_separate_property_section): New function.
>> (xtensa_get_property_section): Invoke
>> xtensa_get_separate_property_section to get individual property
>> section if it exists, common property section otherwise.
>> (xtensa_make_property_section): Pass elf32xtensa_separate_props
>> to xtensa_property_section_name.
>>
>> gas/
>> * config/tc-xtensa.c (elf32xtensa_separate_props): New
>> declaration.
>> (option_separate_props, option_no_separate_props): New
>> enumeration constants.
>> (md_longopts): Add separate-prop-tables option.
>> (md_parse_option): Add cases for option_separate_props and
>> option_no_separate_props.
>> (md_show_usage): Add help for [no-]separate-prop-tables options.
>
> Approved. Thanks.
Thank you. Applied to master.
-- Max