Re: [PATCH] Add __attribute__((malloc) to allocator and remove unused code

2018-05-17 Thread Richard Biener
On Thu, May 17, 2018 at 1:42 PM Marc Glisse wrote: > On Thu, 17 May 2018, Richard Biener wrote: > > On Thu, May 17, 2018 at 1:14 PM Marc Glisse wrote: > > > >> On Thu, 17 May 2018, Jonathan Wakely wrote: > > > >>> On 17/05/18 12:54 +0200, Marc Glisse wrote: > On Mon, 14 May 2018, Jonathan

Re: [PATCH] Add __attribute__((malloc) to allocator and remove unused code

2018-05-17 Thread Marc Glisse
On Thu, 17 May 2018, Richard Biener wrote: On Thu, May 17, 2018 at 1:14 PM Marc Glisse wrote: On Thu, 17 May 2018, Jonathan Wakely wrote: On 17/05/18 12:54 +0200, Marc Glisse wrote: On Mon, 14 May 2018, Jonathan Wakely wrote: As discussed at https://gcc.gnu.org/ml/libstdc++/2018-01/msg0

Re: [PATCH] Add __attribute__((malloc) to allocator and remove unused code

2018-05-17 Thread Richard Biener
On Thu, May 17, 2018 at 1:14 PM Marc Glisse wrote: > On Thu, 17 May 2018, Jonathan Wakely wrote: > > On 17/05/18 12:54 +0200, Marc Glisse wrote: > >> On Mon, 14 May 2018, Jonathan Wakely wrote: > >> > >>> As discussed at https://gcc.gnu.org/ml/libstdc++/2018-01/msg00073.html > >>> we can simplif

Re: [PATCH] Add __attribute__((malloc) to allocator and remove unused code

2018-05-17 Thread Marc Glisse
On Thu, 17 May 2018, Jonathan Wakely wrote: On 17/05/18 12:54 +0200, Marc Glisse wrote: On Mon, 14 May 2018, Jonathan Wakely wrote: As discussed at https://gcc.gnu.org/ml/libstdc++/2018-01/msg00073.html we can simplify the allocator function for valarray memory. I also noticed that the _Array

Re: [PATCH] Add __attribute__((malloc) to allocator and remove unused code

2018-05-17 Thread Jonathan Wakely
On 17/05/18 12:54 +0200, Marc Glisse wrote: On Mon, 14 May 2018, Jonathan Wakely wrote: As discussed at https://gcc.gnu.org/ml/libstdc++/2018-01/msg00073.html we can simplify the allocator function for valarray memory. I also noticed that the _Array(size_t) constructor is never used. *

Re: [PATCH] Add __attribute__((malloc) to allocator and remove unused code

2018-05-17 Thread Marc Glisse
On Mon, 14 May 2018, Jonathan Wakely wrote: As discussed at https://gcc.gnu.org/ml/libstdc++/2018-01/msg00073.html we can simplify the allocator function for valarray memory. I also noticed that the _Array(size_t) constructor is never used. * include/bits/valarray_array.h (__valarray_ge

[PATCH] Add __attribute__((malloc) to allocator and remove unused code

2018-05-14 Thread Jonathan Wakely
As discussed at https://gcc.gnu.org/ml/libstdc++/2018-01/msg00073.html we can simplify the allocator function for valarray memory. I also noticed that the _Array(size_t) constructor is never used. * include/bits/valarray_array.h (__valarray_get_memory): Remove. (__valarray_get_sto