Re: [v3] libstdc++/55979 (+ notes about 55977)

2013-03-18 Thread Paolo Carlini
Hi, On 03/17/2013 06:45 PM, Jonathan Wakely wrote: On 17 March 2013 17:14, Paolo Carlini wrote: I guess we could at least work around the problem by going back to _M_get_Tp_allocator().construct in _M_create_node (or, better, the allocator_traits::construct equivalent, per the recent fix for

[v3] libstdc++/55979 (+ notes about 55977)

2013-03-17 Thread Paolo Carlini
Hi, I had a look to these two PRs and, as regards the former, the below patchlet seems Ok to me even for 4.8.1. About the latter: a similar change works fine for std::vector and std::deque, but std::list for example has another problem, which is already exposed by the v1.emplace_back(i);

Re: [v3] libstdc++/55979 (+ notes about 55977)

2013-03-17 Thread Jonathan Wakely
On 17 March 2013 17:14, Paolo Carlini wrote: I guess we could at least work around the problem by going back to _M_get_Tp_allocator().construct in _M_create_node (or, better, the allocator_traits::construct equivalent, per the recent fix for 56613; we would use it on _Tp actually, everywhere)