Re: [patch] Avoid memory allocations when making futures ready

2014-08-07 Thread Jonathan Wakely
On 08/08/14 00:07 +0200, Dominique Dhumieres wrote: This patch replaces some reference members with pointers, ... ... Tested x86_64-linux, committed to trunk. This breaks bootstrap (see https://gcc.gnu.org/ml/gcc-regression/2014-08/): Fixed by this patch, sorry about that. I was caught out (a

Re: [patch] Avoid memory allocations when making futures ready

2014-08-07 Thread Jonathan Wakely
On 7 August 2014 23:07, Dominique Dhumieres wrote: >> This patch replaces some reference members with pointers, ... >> ... >> Tested x86_64-linux, committed to trunk. > > This breaks bootstrap (see https://gcc.gnu.org/ml/gcc-regression/2014-08/): Strange, I didn't see that - I'll fix it ASAP.

Re: [patch] Avoid memory allocations when making futures ready

2014-08-07 Thread Dominique Dhumieres
> This patch replaces some reference members with pointers, ... > ... > Tested x86_64-linux, committed to trunk. This breaks bootstrap (see https://gcc.gnu.org/ml/gcc-regression/2014-08/): ... libtool: compile: /opt/gcc/build_w/./gcc/xgcc -shared-libgcc -B/opt/gcc/build_w/./gcc -nostdinc++ -L/

[patch] Avoid memory allocations when making futures ready

2014-08-07 Thread Jonathan Wakely
This patch replaces some reference members with pointers, so that the types containing them become trivial and are more efficient to pass as function arguments and can use the small object optimization when stored in a std::function, so that promise::set_value() doesn't need to allocate any memory