Re: [PATCH 4/4] Improve maybe_remove_writeonly_store to do a simple DCE for defining statement

2021-10-20 Thread Andrew Pinski via Gcc-patches
On Wed, Oct 20, 2021 at 12:54 PM Jeff Law via Gcc-patches wrote: > > > > On 10/20/2021 12:58 AM, Richard Biener wrote: > > On Wed, Oct 20, 2021 at 1:14 AM Jeff Law via Gcc-patches > > wrote: > >> > >> > >> On 10/18/2021 10:54 PM, apinski--- via Gcc-patches wrote: > >>> From: Andrew Pinski > >>>

Re: [PATCH 4/4] Improve maybe_remove_writeonly_store to do a simple DCE for defining statement

2021-10-20 Thread Jeff Law via Gcc-patches
On 10/20/2021 12:58 AM, Richard Biener wrote: On Wed, Oct 20, 2021 at 1:14 AM Jeff Law via Gcc-patches wrote: On 10/18/2021 10:54 PM, apinski--- via Gcc-patches wrote: From: Andrew Pinski Instead of putting a full blow DCE after execute_fixup_cfg, it makes sense to try to remove the def

Re: [PATCH 4/4] Improve maybe_remove_writeonly_store to do a simple DCE for defining statement

2021-10-19 Thread Richard Biener via Gcc-patches
On Wed, Oct 20, 2021 at 1:14 AM Jeff Law via Gcc-patches wrote: > > > > On 10/18/2021 10:54 PM, apinski--- via Gcc-patches wrote: > > From: Andrew Pinski > > > > Instead of putting a full blow DCE after execute_fixup_cfg, it makes sense > > to try to remove the defining statement for the store th

Re: [PATCH 4/4] Improve maybe_remove_writeonly_store to do a simple DCE for defining statement

2021-10-19 Thread Jeff Law via Gcc-patches
On 10/18/2021 10:54 PM, apinski--- via Gcc-patches wrote: From: Andrew Pinski Instead of putting a full blow DCE after execute_fixup_cfg, it makes sense to try to remove the defining statement for the store that is being removed. Right now we only handle PHI node statements as there needs no

[PATCH 4/4] Improve maybe_remove_writeonly_store to do a simple DCE for defining statement

2021-10-18 Thread apinski--- via Gcc-patches
From: Andrew Pinski Instead of putting a full blow DCE after execute_fixup_cfg, it makes sense to try to remove the defining statement for the store that is being removed. Right now we only handle PHI node statements as there needs no extra checks except for it is only used once in the store stat