Re: [PATCH] Fix PR86462

2018-07-12 Thread Jakub Jelinek
On Thu, Jul 12, 2018 at 01:01:14PM +0200, Richard Biener wrote: > No, I don't think so. OK, I'll remove the loop over > BLOCK_NON_LOCALIZED_VAR and keep the original check for it. > > I also simplified stuff by hoisting the TREE_USED and friends checks. > > Bootstrap & regtest running on x86_64

Re: [PATCH] Fix PR86462

2018-07-12 Thread Richard Biener
On Thu, 12 Jul 2018, Jakub Jelinek wrote: > On Thu, Jul 12, 2018 at 12:29:20PM +0200, Richard Biener wrote: > > After my PR86413 fix to always annotate existing lexical block DIEs with > > range attributes debuginfo grows significantly in case we previously > > had "stale" lexical block DIEs witho

Re: [PATCH] Fix PR86462

2018-07-12 Thread Jakub Jelinek
On Thu, Jul 12, 2018 at 12:29:20PM +0200, Richard Biener wrote: > After my PR86413 fix to always annotate existing lexical block DIEs with > range attributes debuginfo grows significantly in case we previously > had "stale" lexical block DIEs without any variables. > > The following fixes this by

[PATCH] Fix PR86462

2018-07-12 Thread Richard Biener
After my PR86413 fix to always annotate existing lexical block DIEs with range attributes debuginfo grows significantly in case we previously had "stale" lexical block DIEs without any variables. The following fixes this by eliding those comletely and not emitting a lexical block DIE for blocks