Re: [PATCH] forwprop: Change optimize_agr_copyprop into forward walk instead of backwards

2025-06-10 Thread Andrew Pinski
On Tue, Jun 10, 2025 at 3:47 AM Richard Biener wrote: > > On Tue, Jun 10, 2025 at 2:02 AM Andrew Pinski wrote: > > > > On Mon, Jun 9, 2025 at 2:49 AM Richard Biener > > wrote: > > > > > > On Sun, Jun 8, 2025 at 7:52 PM Andrew Pinski > > > wrote: > > > > > > > > While thinking about how to impl

Re: [PATCH] forwprop: Change optimize_agr_copyprop into forward walk instead of backwards

2025-06-10 Thread Richard Biener
On Tue, Jun 10, 2025 at 2:02 AM Andrew Pinski wrote: > > On Mon, Jun 9, 2025 at 2:49 AM Richard Biener > wrote: > > > > On Sun, Jun 8, 2025 at 7:52 PM Andrew Pinski > > wrote: > > > > > > While thinking about how to implement the rest of the copy prop and makes > > > sure not > > > to introduc

Re: [PATCH] forwprop: Change optimize_agr_copyprop into forward walk instead of backwards

2025-06-09 Thread Andrew Pinski
On Mon, Jun 9, 2025 at 2:49 AM Richard Biener wrote: > > On Sun, Jun 8, 2025 at 7:52 PM Andrew Pinski wrote: > > > > While thinking about how to implement the rest of the copy prop and makes > > sure not > > to introduce some compile time problems, optimize_agr_copyprop should be > > changed >

Re: [PATCH] forwprop: Change optimize_agr_copyprop into forward walk instead of backwards

2025-06-09 Thread Richard Biener
On Sun, Jun 8, 2025 at 7:52 PM Andrew Pinski wrote: > > While thinking about how to implement the rest of the copy prop and makes > sure not > to introduce some compile time problems, optimize_agr_copyprop should be > changed > into a forwproping rather than looking backwards. Can you explain t

[PATCH] forwprop: Change optimize_agr_copyprop into forward walk instead of backwards

2025-06-08 Thread Andrew Pinski
While thinking about how to implement the rest of the copy prop and makes sure not to introduce some compile time problems, optimize_agr_copyprop should be changed into a forwproping rather than looking backwards. Bootstrapped and tested on x86_64-linux-gnu. gcc/ChangeLog: * tree-ssa-fo