diff mbox series

[2/2] ipa: Move pass_ipa_cdtor_merge before pass_ipa_cp and pass_ipa_sra

Message ID ri68qxobc6g.fsf@virgil.suse.cz
State New
Headers show
Series [1/2] ipa: Treat static constructors and destructors as non-local (PR 115815) | expand

Commit Message

Martin Jambor July 26, 2024, 11:55 a.m. UTC
Hi,

when looking at PR 115815 we realized that it would make sense to make
calls to functions originally declared static constructors and
destructors created by pass_ipa_cdtor_merge visible to IPA-SRA.  This
patch does that.

Bootstrapped and tested on x86_64-linux.  OK for master?

Thanks,

Martin


gcc/ChangeLog:

2024-07-25  Martin Jambor  <mjambor@suse.cz>

	* passes.def: Move pass_ipa_cdtor_merge before pass_ipa_cp and
	pass_ipa_sra.
---
 gcc/passes.def | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Martin Jambor Aug. 9, 2024, 12:03 p.m. UTC | #1
Hello,

and ping please.

Martin

On Fri, Jul 26 2024, Martin Jambor wrote:
> Hi,
>
> when looking at PR 115815 we realized that it would make sense to make
> calls to functions originally declared static constructors and
> destructors created by pass_ipa_cdtor_merge visible to IPA-SRA.  This
> patch does that.
>
> Bootstrapped and tested on x86_64-linux.  OK for master?
>
> Thanks,
>
> Martin
>
>
> gcc/ChangeLog:
>
> 2024-07-25  Martin Jambor  <mjambor@suse.cz>
>
> 	* passes.def: Move pass_ipa_cdtor_merge before pass_ipa_cp and
> 	pass_ipa_sra.
> ---
>  gcc/passes.def | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gcc/passes.def b/gcc/passes.def
> index b06d6d45f63..33b2c10c9c9 100644
> --- a/gcc/passes.def
> +++ b/gcc/passes.def
> @@ -157,9 +157,9 @@ along with GCC; see the file COPYING3.  If not see
>    NEXT_PASS (pass_ipa_profile);
>    NEXT_PASS (pass_ipa_icf);
>    NEXT_PASS (pass_ipa_devirt);
> +  NEXT_PASS (pass_ipa_cdtor_merge);
>    NEXT_PASS (pass_ipa_cp);
>    NEXT_PASS (pass_ipa_sra);
> -  NEXT_PASS (pass_ipa_cdtor_merge);
>    NEXT_PASS (pass_ipa_fn_summary);
>    NEXT_PASS (pass_ipa_inline);
>    NEXT_PASS (pass_ipa_pure_const);
> -- 
> 2.45.2
Martin Jambor Aug. 27, 2024, 7:01 a.m. UTC | #2
Hello,

and ping please.

Martin

On Fri, Aug 09 2024, Martin Jambor wrote:
> Hello,
>
> and ping please.
>
> Martin
>
> On Fri, Jul 26 2024, Martin Jambor wrote:
>> Hi,
>>
>> when looking at PR 115815 we realized that it would make sense to make
>> calls to functions originally declared static constructors and
>> destructors created by pass_ipa_cdtor_merge visible to IPA-SRA.  This
>> patch does that.
>>
>> Bootstrapped and tested on x86_64-linux.  OK for master?
>>
>> Thanks,
>>
>> Martin
>>
>>
>> gcc/ChangeLog:
>>
>> 2024-07-25  Martin Jambor  <mjambor@suse.cz>
>>
>> 	* passes.def: Move pass_ipa_cdtor_merge before pass_ipa_cp and
>> 	pass_ipa_sra.
>> ---
>>  gcc/passes.def | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/gcc/passes.def b/gcc/passes.def
>> index b06d6d45f63..33b2c10c9c9 100644
>> --- a/gcc/passes.def
>> +++ b/gcc/passes.def
>> @@ -157,9 +157,9 @@ along with GCC; see the file COPYING3.  If not see
>>    NEXT_PASS (pass_ipa_profile);
>>    NEXT_PASS (pass_ipa_icf);
>>    NEXT_PASS (pass_ipa_devirt);
>> +  NEXT_PASS (pass_ipa_cdtor_merge);
>>    NEXT_PASS (pass_ipa_cp);
>>    NEXT_PASS (pass_ipa_sra);
>> -  NEXT_PASS (pass_ipa_cdtor_merge);
>>    NEXT_PASS (pass_ipa_fn_summary);
>>    NEXT_PASS (pass_ipa_inline);
>>    NEXT_PASS (pass_ipa_pure_const);
>> -- 
>> 2.45.2
diff mbox series

Patch

diff --git a/gcc/passes.def b/gcc/passes.def
index b06d6d45f63..33b2c10c9c9 100644
--- a/gcc/passes.def
+++ b/gcc/passes.def
@@ -157,9 +157,9 @@  along with GCC; see the file COPYING3.  If not see
   NEXT_PASS (pass_ipa_profile);
   NEXT_PASS (pass_ipa_icf);
   NEXT_PASS (pass_ipa_devirt);
+  NEXT_PASS (pass_ipa_cdtor_merge);
   NEXT_PASS (pass_ipa_cp);
   NEXT_PASS (pass_ipa_sra);
-  NEXT_PASS (pass_ipa_cdtor_merge);
   NEXT_PASS (pass_ipa_fn_summary);
   NEXT_PASS (pass_ipa_inline);
   NEXT_PASS (pass_ipa_pure_const);