This is the mail archive of the mailing list for the GCC 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] Generate a label for the split cold function while using -freorder-blocks-and-partition

On Tue, Apr 23, 2013 at 9:59 PM, Jakub Jelinek <> wrote:
> On Tue, Apr 23, 2013 at 03:58:06PM -0700, Sriraman Tallam wrote:
>>   This patch generates labels for cold function parts that are split when
>> using the option -freorder-blocks-and-partition.  The cold label name
>> is generated by suffixing ".cold" to the assembler name of the hot
>> function.
>>   This is useful when getting back traces from gdb when the cold function
>> part does get executed.
>>         * final.c (final_scan_insn): Generate cold label name by suffixing
>>         ".cold" to function's assembler name.
>>         * gcc.dg/tree-prof/cold_partition_label.c: New test.
> This doesn't honor NO_DOT_IN_LABEL (and NO_DOLLAR_IN_LABEL).

Fixed, by calling clean_symbol_name

> Also, don't some function start in cold section and then switch into hot
> section?

I am not able to generate a test where this happens. However, I  fixed
this problem by generating the cold label only when the first function
block is not cold.

Patch attached, please see if this is ok.


>         Jakub

Attachment: split_code_label_patch.txt
Description: Text document

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