Use version namespace in normal mode

Jonathan Wakely
Thu Oct 6 16:06:00 GMT 2016

On 01/10/16 14:22 +0200, François Dumont wrote:
>On 30/09/2016 17:40, Jonathan Wakely wrote:
>>On 29/09/16 21:59 +0200, François Dumont wrote:
>>>   I think _GLIBCXX_BEGIN_NAMESPACE_ALGO should default to 
>>>_GLIBCXX_BEGIN_NAMESPACE_VERSION when parallel mode is not active. 
>>Hmm, yes, I think this is correct, otherwise we're missing the VERSION
>>namespace in normal mode. But it seems we've always been missing it
>>since those macros were introduced in GCC 4.6 so I'd like to
>>investigate the consequences for gnu-versioned-namespace more
>>>   * include/bits/c++config (_GLIBCXX_BEGIN_NAMESPACE_ALGO)
>>>   (_GLIBCXX_END_NAMESPACE_ALGO): Default to respectively
>>>   when parallel mode is not active.
>>>   Likewise.
>>>   Ok to commit after normal check ? Should I rebuild library with 
>>>versioned namespace activated ?
>>Any change affecting the NAMESPACE_VERSION macros should be tested
>>with the gnu-versioned-namespace, since those macros are only used for
>>that mode.
>>But it can't be tested currently, see PR 77794. So let's wait until
>>I've fixed 77794, when we can test it.
>The reason I am proposing this is that if my debug algos patch is 
>applied a lot of algos won't be in versionned namespace anymore.
>Broken for broken I might apply it even if not really tested, no ?

It can be tested now, I've fixed PR 77794.

More information about the Gcc-patches mailing list