Re: [C++ Patch] Remove quick fix for c++/85553

2018-10-24 Thread Jason Merrill
On 10/17/18 3:38 PM, Paolo Carlini wrote: Hi Jakub, On 17/10/18 19:42, Jakub Jelinek wrote: On Wed, Oct 17, 2018 at 07:20:53PM +0200, Paolo Carlini wrote: Hi, as you probably remember, very close to the release of 8.1.0 we noticed that my fix for c++/70808 was causing c++/85553, which Jakub

Re: [C++ Patch] Remove quick fix for c++/85553

2018-10-17 Thread Paolo Carlini
Hi Jakub, On 17/10/18 19:42, Jakub Jelinek wrote: On Wed, Oct 17, 2018 at 07:20:53PM +0200, Paolo Carlini wrote: Hi, as you probably remember, very close to the release of 8.1.0 we noticed that my fix for c++/70808 was causing c++/85553, which Jakub promptly fixed. However, we later found out

Re: [C++ Patch] Remove quick fix for c++/85553

2018-10-17 Thread Jakub Jelinek
On Wed, Oct 17, 2018 at 07:20:53PM +0200, Paolo Carlini wrote: > Hi, > > as you probably remember, very close to the release of 8.1.0 we noticed that > my fix for c++/70808 was causing c++/85553, which Jakub promptly fixed. > However, we later found out that the real problem was a latent issue in

[C++ Patch] Remove quick fix for c++/85553

2018-10-17 Thread Paolo Carlini
Hi, as you probably remember, very close to the release of 8.1.0 we noticed that my fix for c++/70808 was causing c++/85553, which Jakub promptly fixed. However, we later found out that the real problem was a latent issue in convert, which I fixed in r259966. Thus, I think that in current tru