Re: [PATCH] DSE: add remove_unused_locals to the todos [PR117096]

2024-10-13 Thread Richard Biener
On Sun, Oct 13, 2024 at 12:07 AM Andrew Pinski wrote: > > This is a better patch to fix PR 117096 (phiopt vs clobbers). > The only time we remove clobbers of local variables is during > remove_unused_locals. Since DSE might remove all of the stores > to a local variable, it makes sense to also try

[PATCH] DSE: add remove_unused_locals to the todos [PR117096]

2024-10-12 Thread Andrew Pinski
This is a better patch to fix PR 117096 (phiopt vs clobbers). The only time we remove clobbers of local variables is during remove_unused_locals. Since DSE might remove all of the stores to a local variable, it makes sense to also try to remove unused local variables afterwards. This shows up more