This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [Patch] [libstdc++] [C++0x] Export std::string base object move constructor.

From: Benjamin Kosnik <bkoz at redhat dot com>
To: Jakub Jelinek <jakub at redhat dot com>
Cc: Adam Butcher <dev dot lists at jessamine dot co dot uk>, gcc-patches at gcc dot gnu dot org, libstdc++ at gcc dot gnu dot org
Date: Tue, 22 Mar 2011 15:57:44 +0100
Subject: Re: [Patch] [libstdc++] [C++0x] Export std::string base object move constructor.
References: <7db3e0ce41aacbeabd8c003deadabb98.squirrel@webmail.plus.net> <20110322133951.GF18914@tyan-ft48-01.lab.bos.redhat.com> <20110322152621.466dc9c6@shotwell> <20110322144754.GH18914@tyan-ft48-01.lab.bos.redhat.com>

> Here is updated patch with testcases, tested with unpatched libstdc++
> where it failed to link and patched where it succeeded.
> The testcases are copies of corresponding moveable.cc tests, just
> using the new class inheriting from std::basic_string<char> resp.
> std::basic_string<wchar_t> and with -fno-inline.
> Is that ok?
> 
> 2011-03-22  Jakub Jelinek  <jakub@redhat.com>
> 
> 	* config/abi/pre/gnu.ver (GLIBCXX_3.4.15): Export _ZNSsC2EOSs
> 	and _ZNSbIwSt11char_traitsIwESaIwEEC2EOS2_.
> 	* config/abi/post/solaris2.8/baseline_symbols.txt:
> Regenerated.
> 	* config/abi/post/solaris2.8/sparcv9/baseline_symbols.txt:
> Likewise.
> 	* config/abi/post/s390x-linux-gnu/baseline_symbols.txt:
> Likewise.
> 	* config/abi/post/x86_64-linux-gnu/baseline_symbols.txt:
> Likewise.
> 	* config/abi/post/x86_64-linux-gnu/32/baseline_symbols.txt:
> Likewise.
> 	* config/abi/post/powerpc-linux-gnu/baseline_symbols.txt:
> Likewise.
> 	* config/abi/post/powerpc64-linux-gnu/baseline_symbols.txt:
> Likewise.
> 	* config/abi/post/powerpc64-linux-gnu/32/baseline_symbols.txt:
> 	Likewise.
> 	* config/abi/post/solaris2.10/baseline_symbols.txt: Likewise.
> 	* config/abi/post/solaris2.10/amd64/baseline_symbols.txt:
> Likewise.
> 	* config/abi/post/solaris2.10/sparcv9/baseline_symbols.txt:
> Likewise.
> 	* config/abi/post/i486-linux-gnu/baseline_symbols.txt:
> Likewise.
> 	* config/abi/post/i386-linux-gnu/baseline_symbols.txt:
> Likewise.
> 	* config/abi/post/s390-linux-gnu/baseline_symbols.txt:
> Likewise.
> 	* testsuite/21_strings/basic_string/cons/char/moveable2.cc:
> New test.
> 	*
> testsuite/21_strings/basic_string/cons/wchar_t/moveable2.cc: New test.

OK. 

Thanks!

-benjamin

References:
- [Patch] [libstdc++] [C++0x] Export std::string base object move constructor.
  - From: Adam Butcher
- Re: [Patch] [libstdc++] [C++0x] Export std::string base object move constructor.
  - From: Jakub Jelinek
- Re: [Patch] [libstdc++] [C++0x] Export std::string base object move constructor.
  - From: Benjamin Kosnik
- Re: [Patch] [libstdc++] [C++0x] Export std::string base object move constructor.
  - From: Jakub Jelinek

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]