Re: [PATCH] Use cxx11 abi in versioned namespace

2023-01-16 Thread Jonathan Wakely via Gcc-patches
On Mon, 16 Jan 2023 at 17:47, François Dumont wrote: > > On 13/01/23 17:33, Jonathan Wakely wrote: > > On Mon, 5 Dec 2022 at 21:14, François Dumont via Libstdc++ > > wrote: > >> I just rebased this patch. > >> > >> All good apart from the to_chars/from_chars symbols issue. > >> > >> François >

Re: [PATCH] Use cxx11 abi in versioned namespace

2023-01-16 Thread François Dumont via Gcc-patches
On 13/01/23 18:15, Jonathan Wakely wrote: @@ -396,7 +376,7 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION // Non-inline namespace for components replaced by alternates in active mode. namespace __cxx1998 { -# if _GLIBCXX_USE_CXX11_ABI +# if _GLIBCXX_USE_CXX11_ABI && !

Re: [PATCH] Use cxx11 abi in versioned namespace

2023-01-16 Thread François Dumont via Gcc-patches
On 13/01/23 18:06, Jonathan Wakely wrote: On Fri, 13 Jan 2023 at 16:33, Jonathan Wakely wrote: On Mon, 5 Dec 2022 at 21:14, François Dumont via Libstdc++ wrote: I just rebased this patch. All good apart from the to_chars/from_chars symbols issue. François On 11/10/22 19:28, François

Re: [PATCH] Use cxx11 abi in versioned namespace

2023-01-16 Thread François Dumont via Gcc-patches
On 13/01/23 17:33, Jonathan Wakely wrote: On Mon, 5 Dec 2022 at 21:14, François Dumont via Libstdc++ wrote: I just rebased this patch. All good apart from the to_chars/from_chars symbols issue. François On 11/10/22 19:28, François Dumont wrote: Hi Now that pretty printer is fixed

Re: [PATCH] Use cxx11 abi in versioned namespace

2023-01-13 Thread Jonathan Wakely via Gcc-patches
@@ -396,7 +376,7 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION // Non-inline namespace for components replaced by alternates in active mode. namespace __cxx1998 { -# if _GLIBCXX_USE_CXX11_ABI +# if _GLIBCXX_USE_CXX11_ABI && ! _GLIBCXX_VERSION_NAMESPACE This should be INLINE not VERSION, right?

Re: [PATCH] Use cxx11 abi in versioned namespace

2023-01-13 Thread Jonathan Wakely via Gcc-patches
On Fri, 13 Jan 2023 at 16:33, Jonathan Wakely wrote: > > On Mon, 5 Dec 2022 at 21:14, François Dumont via Libstdc++ > wrote: > > > > I just rebased this patch. > > > > All good apart from the to_chars/from_chars symbols issue. > > > > François > > > > > > On 11/10/22 19:28, François Dumont

Re: [PATCH] Use cxx11 abi in versioned namespace

2023-01-13 Thread Jonathan Wakely via Gcc-patches
On Mon, 5 Dec 2022 at 21:14, François Dumont via Libstdc++ wrote: > > I just rebased this patch. > > All good apart from the to_chars/from_chars symbols issue. > > François > > > On 11/10/22 19:28, François Dumont wrote: > > Hi > > > > Now that pretty printer is fixed (once patch validated)

Re: [PATCH] Use cxx11 abi in versioned namespace

2022-12-05 Thread François Dumont via Gcc-patches
I just rebased this patch. All good apart from the to_chars/from_chars symbols issue. François On 11/10/22 19:28, François Dumont wrote: Hi     Now that pretty printer is fixed (once patch validated) I'd like to propose this patch again.     Note that I'am adding a check on pretty

Re: [PATCH] Use cxx11 abi in versioned namespace

2022-10-11 Thread François Dumont via Gcc-patches
Hi     Now that pretty printer is fixed (once patch validated) I'd like to propose this patch again.     Note that I'am adding a check on pretty printer with a std::any on a std::wstring. I did so because of the FIXME in printers.py which is dealing with 'std::string' explicitely. Looks

[PATCH] Use cxx11 abi in versioned namespace

2022-05-30 Thread François Dumont via Gcc-patches
Hi Here is the patch to use cxx11 abi for the versioned namespace mode. It is still suffering from a side effect on gdb pretty printers with 3 errors like: got: type = std::unique_ptrstd::__8::char_traits, std::__8::allocator >>[]>>[99]> FAIL: libstdc++-prettyprinters/80276.cc whatis p4