Re: [PATCH] Add INCLUDE_UNIQUE_PTR and use it (PR bootstrap/82610)

2017-10-23 Thread Richard Biener
On October 23, 2017 8:15:20 PM GMT+02:00, David Malcolm wrote: >On Mon, 2017-10-23 at 16:40 +0100, Pedro Alves wrote: >> On 10/23/2017 04:17 PM, Jonathan Wakely wrote: >> > On 23/10/17 17:07 +0200, Michael Matz wrote: >> > > Hi, >> > > >> > > On Mon, 23 Oct 2017, Richard Biener wrote: >> > > >>

Re: [PATCH] Add INCLUDE_UNIQUE_PTR and use it (PR bootstrap/82610)

2017-10-23 Thread David Malcolm
On Mon, 2017-10-23 at 21:43 +0200, Gerald Pfeifer wrote: > On Mon, 23 Oct 2017, David Malcolm wrote: > > Here's a different patch, which instead moves the include of our > > "unique-ptr.h" to system.h (conditionalized on INCLUDE_UNIQUE_PTR), > > after the decl of "free" and before the redefinition

Re: [PATCH] Add INCLUDE_UNIQUE_PTR and use it (PR bootstrap/82610)

2017-10-23 Thread Gerald Pfeifer
On Mon, 23 Oct 2017, David Malcolm wrote: > Here's a different patch, which instead moves the include of our > "unique-ptr.h" to system.h (conditionalized on INCLUDE_UNIQUE_PTR), > after the decl of "free" and before the redefinition of "abort". Thanks for your persistance in tackling this, David!

Re: [PATCH] Add INCLUDE_UNIQUE_PTR and use it (PR bootstrap/82610)

2017-10-23 Thread Pedro Alves
On 10/23/2017 07:15 PM, David Malcolm wrote: > OK for trunk? FAOD, FWIW, LGTM. Thanks, Pedro Alves

[PATCH] Add INCLUDE_UNIQUE_PTR and use it (PR bootstrap/82610)

2017-10-23 Thread David Malcolm
On Mon, 2017-10-23 at 16:40 +0100, Pedro Alves wrote: > On 10/23/2017 04:17 PM, Jonathan Wakely wrote: > > On 23/10/17 17:07 +0200, Michael Matz wrote: > > > Hi, > > > > > > On Mon, 23 Oct 2017, Richard Biener wrote: > > > > > > > I guess so. But we have to make gdb happy as well. It really > > >