Re: [PATCH] Maybe fix PR87134

2018-09-17 Thread Rainer Orth
Hi Iain, > Hi Rainer, > >> On 6 Sep 2018, at 21:21, Rainer Orth wrote: >> > >>> I can confirm the same, repeatable, fail on i686-darwin10 (and it >>> reproduces with -save-temps) >>> (and the vNULL change does not fix it there either) - don’t have access to >>> the machine now until >>> later in

Re: [PATCH] Maybe fix PR87134

2018-09-17 Thread Iain Sandoe
Hi Rainer, > On 6 Sep 2018, at 21:21, Rainer Orth wrote: > >> I can confirm the same, repeatable, fail on i686-darwin10 (and it >> reproduces with -save-temps) >> (and the vNULL change does not fix it there either) - don’t have access to >> the machine now until >> later in the month tho. > >

Re: [PATCH] Maybe fix PR87134

2018-09-06 Thread Rainer Orth
Hi Iain, > I can confirm the same, repeatable, fail on i686-darwin10 (and it > reproduces with -save-temps) > (and the vNULL change does not fix it there either) - don’t have access to > the machine now until > later in the month tho. same here on i386-apple-darwin11, and I can easily fire off te

Re: [PATCH] Maybe fix PR87134

2018-09-06 Thread Iain Sandoe
> On 6 Sep 2018, at 17:24, Gerald Pfeifer wrote: > > On Wed, 5 Sep 2018, Richard Biener wrote: >> So I'm testing the following then, leaving the placement new untouched >> (no init is fine) and then assign from vNULL. >> >> 2018-09-05 Richard Biener >> >> PR bootstrap/87134 >> *

Re: [PATCH] Maybe fix PR87134

2018-09-06 Thread Gerald Pfeifer
On Wed, 5 Sep 2018, Richard Biener wrote: > So I'm testing the following then, leaving the placement new untouched > (no init is fine) and then assign from vNULL. > > 2018-09-05 Richard Biener > > PR bootstrap/87134 > * tree-ssa-sccvn.c (rpo_elim::eliminate_push_avail): Make sure >

Re: [PATCH] Maybe fix PR87134

2018-09-05 Thread Richard Biener
On Wed, 5 Sep 2018, Richard Biener wrote: > On Wed, 5 Sep 2018, Jakub Jelinek wrote: > > > On Wed, Sep 05, 2018 at 12:55:55PM +0200, Richard Biener wrote: > > > > > > The following makes sure to call the default CTOR when emplacing a > > > vec<> in the avail hash-map. Certainly the intent was t

Re: [PATCH] Maybe fix PR87134

2018-09-05 Thread Richard Biener
On Wed, 5 Sep 2018, Jakub Jelinek wrote: > On Wed, Sep 05, 2018 at 12:55:55PM +0200, Richard Biener wrote: > > > > The following makes sure to call the default CTOR when emplacing a > > vec<> in the avail hash-map. Certainly the intent was to zero-initialize > > the m_vec member. > > Guess it w

Re: [PATCH] Maybe fix PR87134

2018-09-05 Thread Jakub Jelinek
On Wed, Sep 05, 2018 at 12:55:55PM +0200, Richard Biener wrote: > > The following makes sure to call the default CTOR when emplacing a > vec<> in the avail hash-map. Certainly the intent was to zero-initialize > the m_vec member. Guess it would be nice to see what older versions of system g++ do