Re: [PATCH] Fix bootstrap miscompare by STV (PR91580)

2019-08-29 Thread Richard Biener
On Thu, 29 Aug 2019, Richard Biener wrote: > On Thu, 29 Aug 2019, Jakub Jelinek wrote: > > > On Thu, Aug 29, 2019 at 12:04:53PM +0200, Richard Biener wrote: > > > + else > > > > Perhaps use > > else if (MAY_HAVE_DEBUG_BIND_INSNS) > > instead so that you don't walk it once again if there can'

Re: [PATCH] Fix bootstrap miscompare by STV (PR91580)

2019-08-29 Thread Richard Biener
On Thu, 29 Aug 2019, Uros Bizjak wrote: > On Thu, Aug 29, 2019 at 12:04 PM Richard Biener wrote: > > > > > > The following fixes the bootstrap-debug miscompare caused by STV > > where we ended up with chain-to-scalar copies just because of > > debug uses. Instead we have to avoid that, eventuall

Re: [PATCH] Fix bootstrap miscompare by STV (PR91580)

2019-08-29 Thread Richard Biener
On Thu, 29 Aug 2019, Jakub Jelinek wrote: > On Thu, Aug 29, 2019 at 12:04:53PM +0200, Richard Biener wrote: > > + else > > Perhaps use > else if (MAY_HAVE_DEBUG_BIND_INSNS) > instead so that you don't walk it once again if there can't be DEBUG_INSNs? Sure - will do as followup to unbreak

Re: [PATCH] Fix bootstrap miscompare by STV (PR91580)

2019-08-29 Thread Jakub Jelinek
On Thu, Aug 29, 2019 at 12:04:53PM +0200, Richard Biener wrote: > + else Perhaps use else if (MAY_HAVE_DEBUG_BIND_INSNS) instead so that you don't walk it once again if there can't be DEBUG_INSNs? Jakub

Re: [PATCH] Fix bootstrap miscompare by STV (PR91580)

2019-08-29 Thread Uros Bizjak
On Thu, Aug 29, 2019 at 12:04 PM Richard Biener wrote: > > > The following fixes the bootstrap-debug miscompare caused by STV > where we ended up with chain-to-scalar copies just because of > debug uses. Instead we have to avoid that, eventually substituting > into debug uses or resetting debug s

[PATCH] Fix bootstrap miscompare by STV (PR91580)

2019-08-29 Thread Richard Biener
The following fixes the bootstrap-debug miscompare caused by STV where we ended up with chain-to-scalar copies just because of debug uses. Instead we have to avoid that, eventually substituting into debug uses or resetting debug stmts when there are reaching defs from both inside and outside of