Re: [PATCH] middle-end/118801 - excessive redundant DEBUG BEGIN_STMT

2025-03-06 Thread Alexandre Oliva
On Feb 11, 2025, Richard Biener wrote: > Alex, is this OK for trunk? >> PR middle-end/118801 >> * tree-ssa-dce.cc (eliminate_unnecessary_stmts): Prune >> sequences of uninterrupted DEBUG BEGIN_STMTs, keeping only >> the last of a set with unique location. It seems like a reasonable trade-off.

Re: [PATCH] middle-end/118801 - excessive redundant DEBUG BEGIN_STMT

2025-02-26 Thread Jakub Jelinek
On Wed, Feb 26, 2025 at 12:36:15PM +0100, Richard Biener wrote: > > Bootstrapped and tested on x86_64-unknown-linux-gnu without > > regressions this time. > > > > Alex, is this OK for trunk? > > Ping. I'd really like to hear from Alex on this one. Jakub

Re: [PATCH] middle-end/118801 - excessive redundant DEBUG BEGIN_STMT

2025-02-26 Thread Richard Biener
On Tue, 11 Feb 2025, Richard Biener wrote: > On Mon, 10 Feb 2025, Richard Biener wrote: > > > On Mon, 10 Feb 2025, Richard Biener wrote: > > > > > The following addresses the fact that we keep an excessive amount of > > > redundant DEBUG BEGIN_STMTs - in the testcase it sums up to 99.999% > > >

Re: [PATCH] middle-end/118801 - excessive redundant DEBUG BEGIN_STMT

2025-02-11 Thread Richard Biener
On Mon, 10 Feb 2025, Richard Biener wrote: > On Mon, 10 Feb 2025, Richard Biener wrote: > > > The following addresses the fact that we keep an excessive amount of > > redundant DEBUG BEGIN_STMTs - in the testcase it sums up to 99.999% > > of all stmts, sucking up compile-time in IL walks. The pa

Re: [PATCH] middle-end/118801 - excessive redundant DEBUG BEGIN_STMT

2025-02-10 Thread Richard Biener
On Mon, 10 Feb 2025, Richard Biener wrote: > The following addresses the fact that we keep an excessive amount of > redundant DEBUG BEGIN_STMTs - in the testcase it sums up to 99.999% > of all stmts, sucking up compile-time in IL walks. The patch amends > the GIMPLE DCE code that elides redundant