Ping

> On 3 Mar 2026, at 8:44 AM, Prachi Godbole <[email protected]> wrote:
> 
> Ping
> 
>> On 24 Feb 2026, at 12:28 PM, Prachi Godbole <[email protected]> wrote:
>> 
>> Ping
>> 
>>> On 27 Jan 2026, at 1:05 PM, Prachi Godbole <[email protected]> wrote:
>>> 
>>> Hi,
>>> 
>>> I’ve updated the patch so that -fipa-reorder-for-locality now uses 
>>> tp_first_run instead of a custom field.
>>> 
>>> This patch adds support for explicit function reordering for
>>> -fipa-reorder-for-locality. This is done by setting tp_first_run for each
>>> processed function in ipa locality cloning pass.  All functions are then 
>>> sorted
>>> using tp_first_run in expand_all_functions ().
>>> 
>>> Bootstrapped and regtested on aarch64-linux-gnu.
>>> Ok for mainline?
>>> 
>>> Signed-off-by: Prachi Godbole [email protected]
>>> 
>>> gcc/ChangeLog:
>>> 
>>>       * cgraphunit.cc (tp_first_run_node_cmp): Update condition.
>>>       (expand_all_functions): Ditto.
>>>       * ipa-locality-cloning.cc (get_default_locality_order): New function.
>>>       (add_node_to_partition): Update tp_first_run.
>>> 
>>> 
>>>> On 13 Jan 2026, at 3:26 PM, Prachi Godbole <[email protected]> wrote:
>>>> 
>>>> As discussed, the binutils/linker changes are present since this commit -
>>>> https://sourceware.org/git/?p=binutils-gdb.git;a=commit;h=5fa5f8f5fe494ba4fe98c11899a5464cd164ec75
>>>> 
>>>>> On 6 Jan 2026, at 2:46 PM, Prachi Godbole <[email protected]> wrote:
>>>>> 
>>>>> Ping
>>>>> 
>>>>>> On 18 Dec 2025, at 9:03 PM, Prachi Godbole <[email protected]> wrote:
>>>>>> 
>>>>>> A gentle ping
>>>>>> 
>>>>>>> On 9 Dec 2025, at 3:11 PM, Prachi Godbole <[email protected]> wrote:
>>>>>>> 
>>>>>>> Ping
>>>>>>> 
>>>>>>>> On 30 Oct 2025, at 10:48 AM, Prachi Godbole <[email protected]> 
>>>>>>>> wrote:
>>>>>>>> 
>>>>>>>> Ping
>>>>>>>> 
>>>>>>>>> On 9 Oct 2025, at 9:10 PM, Prachi Godbole <[email protected]> wrote:
>>>>>>>>> 
>>>>>>>>> This patch adds support to linker driven function reordering by 
>>>>>>>>> creating function sections with a prefix '.text.sorted' which are 
>>>>>>>>> then sorted using SORT in by the linker. Appropriate support for this 
>>>>>>>>> is already present in binutils.
>>>>>>>>> 
>>>>>>>>> Bootstrapped and regtested on aarch64-linux-gnu. Ok for mainline?
>>>>>>>>> 
>>>>>>>>> Thanks,
>>>>>>>>> Prachi
>>>>>>>>> 
>>>>>>>>> Signed-off-by: Prachi Godbole [email protected]
>>>>>>>>> 
>>>>>>>>> gcc/ChangeLog:
>>>>>>>>> 
>>>>>>>>> * cgraph.cc (cgraph_node::dump): Dump value of text_locality_order.
>>>>>>>>> * cgraph.h (struct cgraph_node): New variable text_locality_order.
>>>>>>>>> * cgraphclones.cc (cgraph_node::create_clone): Initialize
>>>>>>>>> text_locality_order.
>>>>>>>>> * cgraphunit.cc (expand_all_functions): Update text_locality_order.
>>>>>>>>> * ipa-locality-cloning.cc (get_default_locality_order): New function.
>>>>>>>>> (add_node_to_partition): Update text_locality_order.
>>>>>>>>> * lto-cgraph.cc (lto_output_node): Write text_locality_order to file.
>>>>>>>>> (input_node): Read text_locality_order.
>>>>>>>>> * varasm.cc (default_function_section): Create function sections based
>>>>>>>>> on text_locality_order.
>>>>>>>>> 
>>>>>>>>> <0001-PATCH-Use-linker-sections-to-explicitly-order-functi.patch>
>>>>>>>> 
>>>>>>> 
>>>>>> 
>>>>> 
>>>> 
>>> 
>>> <0001-PATCH-Use-tp_first_run-to-explicitly-order-functions.patch>
>> 
> 

Reply via email to