[google] Add SECTION_EXCLUDE flag and exclude .gnu.callgraph sections (issue5126041)

This patch adds a new flag to mark sections as exclude sections. Such
sections will be discarded by the linker. Also, mark .gnu.callgraph
sections which store the callgraph edge profile info as exclude
sections. 

I also want the SECTION_EXCLUDE part alone to be considered for trunk.



--
This patch is available for review at http://codereview.appspot.com/5126041

On 09/23/2011 04:45 PM, Sriraman Tallam wrote:
> I also want the SECTION_EXCLUDE part alone to be considered for trunk.

This is ok for trunk.

Kai, is there a similar flag for pe-coff?  I.e. is there something
reasonable that we can add to i386_pe_asm_named_section?


r~

2011/9/27 Richard Henderson <rth@redhat.com>:
> On 09/23/2011 04:45 PM, Sriraman Tallam wrote:
>> I also want the SECTION_EXCLUDE part alone to be considered for trunk.
>
> This is ok for trunk.
>
> Kai, is there a similar flag for pe-coff?  I.e. is there something
> reasonable that we can add to i386_pe_asm_named_section?

Well, pe-coff also has SEC_EXCLUDE.  Its meaning is that the section
gets discarded on linking.  I checked in gas/config/obj-coff.c and
there is right now AFAICS no way to specify this flag for a section.
But it should be trivial to add it there with specifier 'e'.

Kai

Submitted the SECTION_EXCLUDE part to trunk.

Thanks,
-Sri.

On Tue, Sep 27, 2011 at 10:51 AM, Kai Tietz <ktietz70@googlemail.com> wrote:
>
> 2011/9/27 Richard Henderson <rth@redhat.com>:
> > On 09/23/2011 04:45 PM, Sriraman Tallam wrote:
> >> I also want the SECTION_EXCLUDE part alone to be considered for trunk.
> >
> > This is ok for trunk.
> >
> > Kai, is there a similar flag for pe-coff?  I.e. is there something
> > reasonable that we can add to i386_pe_asm_named_section?
>
> Well, pe-coff also has SEC_EXCLUDE.  Its meaning is that the section
> gets discarded on linking.  I checked in gas/config/obj-coff.c and
> there is right now AFAICS no way to specify this flag for a section.
> But it should be trivial to add it there with specifier 'e'.
>
> Kai

> Index: final.c
> ===================================================================
> --- final.c     (revision 179104)
> +++ final.c     (working copy)
> @@ -4428,7 +4428,7 @@ rest_of_handle_final (void)
>       && cgraph_node (current_function_decl) != NULL
>       && (cgraph_node (current_function_decl))->callees != NULL)
>     {
> -      flags = SECTION_DEBUG;
> +      flags = SECTION_DEBUG | SECTION_EXCLUDE;
>       asprintf (&profile_fnname, ".gnu.callgraph.text.%s", fnname);
>       switch_to_section (get_section (profile_fnname, flags, NULL));
>       fprintf (asm_out_file, "\t.string \"Function %s\"\n", fnname);

This part is OK for the google/gcc-4_6 branch.

-cary