Re: [PATCH] Another BLOCK issue - PR56570

2013-03-08 Thread Richard Biener
On Fri, 8 Mar 2013, Jakub Jelinek wrote: > On Fri, Mar 08, 2013 at 01:59:23PM +0100, Richard Biener wrote: > > This PR shows that when SRA creates DECL_DEBUG_EXPRs with locations > > (and blocks) this does not work with inlining. The reason is that > > the inliner does not bother to copy vars use

Re: [PATCH] Another BLOCK issue - PR56570

2013-03-08 Thread Jakub Jelinek
On Fri, Mar 08, 2013 at 01:59:23PM +0100, Richard Biener wrote: > This PR shows that when SRA creates DECL_DEBUG_EXPRs with locations > (and blocks) this does not work with inlining. The reason is that > the inliner does not bother to copy vars used only in lhs of > debug-stmts (eek) and thus ends

[PATCH] Another BLOCK issue - PR56570

2013-03-08 Thread Richard Biener
This PR shows that when SRA creates DECL_DEBUG_EXPRs with locations (and blocks) this does not work with inlining. The reason is that the inliner does not bother to copy vars used only in lhs of debug-stmts (eek) and thus ends up sharing DECL_DEBUG_EXPRs (and not re-mapping them and their locatio