Re: [PATCH] Move substitute_and_fold over to use simple_dce_from_worklist

2023-06-26 Thread Andrew Pinski via Gcc-patches
On Mon, Jun 26, 2023 at 12:41 PM Andrew Pinski wrote: > > On Mon, Jun 26, 2023 at 11:49 AM Andrew Pinski wrote: > > > > On Mon, Jun 26, 2023 at 9:13 AM Andrew Pinski wrote: > > > > > > On Sun, Jun 25, 2023 at 10:59 PM Jan-Benedict Glaw > > > wrote: > > > > > > > > Hi Andrew, > > > > > > > >

Re: [PATCH] Move substitute_and_fold over to use simple_dce_from_worklist

2023-06-26 Thread Jan-Benedict Glaw
Hi Andrew, On Mon, 2023-06-26 09:13:51 -0700, Andrew Pinski wrote: > On Sun, Jun 25, 2023 at 10:59 PM Jan-Benedict Glaw wrote: > > On Fri, 2023-05-05 08:17:19 -0700, Andrew Pinski via Gcc-patches > > wrote: > > > While looking into a different issue, I noticed that it > > > would take until

Re: [PATCH] Move substitute_and_fold over to use simple_dce_from_worklist

2023-06-26 Thread Andrew Pinski via Gcc-patches
On Mon, Jun 26, 2023 at 11:49 AM Andrew Pinski wrote: > > On Mon, Jun 26, 2023 at 9:13 AM Andrew Pinski wrote: > > > > On Sun, Jun 25, 2023 at 10:59 PM Jan-Benedict Glaw > > wrote: > > > > > > Hi Andrew, > > > > > > On Fri, 2023-05-05 08:17:19 -0700, Andrew Pinski via Gcc-patches > > >

Re: [PATCH] Move substitute_and_fold over to use simple_dce_from_worklist

2023-06-26 Thread Andrew Pinski via Gcc-patches
On Mon, Jun 26, 2023 at 9:13 AM Andrew Pinski wrote: > > On Sun, Jun 25, 2023 at 10:59 PM Jan-Benedict Glaw wrote: > > > > Hi Andrew, > > > > On Fri, 2023-05-05 08:17:19 -0700, Andrew Pinski via Gcc-patches > > wrote: > > > While looking into a different issue, I noticed that it > > > would

Re: [PATCH] Move substitute_and_fold over to use simple_dce_from_worklist

2023-06-26 Thread Andrew Pinski via Gcc-patches
On Sun, Jun 25, 2023 at 10:59 PM Jan-Benedict Glaw wrote: > > Hi Andrew, > > On Fri, 2023-05-05 08:17:19 -0700, Andrew Pinski via Gcc-patches > wrote: > > While looking into a different issue, I noticed that it > > would take until the second forwprop pass to do some > > forward proping and it

Re: [PATCH] Move substitute_and_fold over to use simple_dce_from_worklist

2023-06-25 Thread Jan-Benedict Glaw
Hi Andrew, On Fri, 2023-05-05 08:17:19 -0700, Andrew Pinski via Gcc-patches wrote: > While looking into a different issue, I noticed that it > would take until the second forwprop pass to do some > forward proping and it was because the ssa name was > used more than once but the second

Re: [PATCH] Move substitute_and_fold over to use simple_dce_from_worklist

2023-05-08 Thread Richard Biener via Gcc-patches
On Fri, May 5, 2023 at 5:18 PM Andrew Pinski via Gcc-patches wrote: > > While looking into a different issue, I noticed that it > would take until the second forwprop pass to do some > forward proping and it was because the ssa name was > used more than once but the second statement was > "dead"

[PATCH] Move substitute_and_fold over to use simple_dce_from_worklist

2023-05-05 Thread Andrew Pinski via Gcc-patches
While looking into a different issue, I noticed that it would take until the second forwprop pass to do some forward proping and it was because the ssa name was used more than once but the second statement was "dead" and we don't remove that until much later. So this uses simple_dce_from_worklist