This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PR 59712 patch
- From: Jonathan Wakely <jwakely dot gcc at gmail dot com>
- To: François Dumont <frs dot dumont at gmail dot com>
- Cc: "libstdc++ at gcc dot gnu dot org" <libstdc++ at gcc dot gnu dot org>, gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 15 Jan 2014 11:37:53 +0000
- Subject: Re: PR 59712 patch
- Authentication-results: sourceware.org; auth=none
- References: <52CF1ADB dot 5070901 at gmail dot com> <CAH6eHdT2p5WwRi7ZBp93nHMATTfbrtD+FfFqqXOmX1JRP7R7SQ at mail dot gmail dot com>
On 14 January 2014 14:44, Jonathan Wakely wrote:
> On 9 January 2014 21:55, François Dumont wrote:
>> Hi
>>
>> Here is a patch for this small problem with clang. It is not a blocking
>> issue for the 4.9 release but at the same time it is a rather safe fix so
>> just tell me if I can commit it.
>>
>> All unordered_* tests run under Linux x86_64. I haven't clang installed
>> at the moment so a clang feedback would be appreciated.
>
> I'm going to test this with Clang to be sure it actually fixes the bug
> before we change anything, but it might have to wait for the weekend.
>
> N.B. please change the comment at the top of the patch to say "pool of
> nodes" instead of "pool of node"
This does fix the error with Clang.
Please go ahead and commit it, with the small change to the comment I
pointed out - thanks!