[PATCH] [RTEMS] Always use atomic builtins for libstdc++

Jonathan Wakely jwakely@redhat.com
Thu Sep 22 08:58:00 GMT 2016


On 22/09/16 09:47 +0100, Jonathan Wakely wrote:
>On 22/09/16 09:41 +0200, Sebastian Huber wrote:
>>libstdc++-v3/
>>	* config/cpu/m68k/atomicity.h: Adjust comment.
>>	* acinclude.m4 (GLIBCXX_ENABLE_ATOMIC_BUILTINS): Honor
>>	explicit atomicity_dir setup via configure.host.
>>	* configure.host (rtems-*): Set atomicity_dir.
>>	* configure: Regenerate.
>>---
>>libstdc++-v3/acinclude.m4                |  5 +++--
>>libstdc++-v3/config/cpu/m68k/atomicity.h |  3 +++
>>libstdc++-v3/configure                   | 11 ++++++-----
>>libstdc++-v3/configure.host              |  4 ++++
>>4 files changed, 16 insertions(+), 7 deletions(-)
>>
>>diff --git a/libstdc++-v3/acinclude.m4 b/libstdc++-v3/acinclude.m4
>>index 6d897be..3256ce4 100644
>>--- a/libstdc++-v3/acinclude.m4
>>+++ b/libstdc++-v3/acinclude.m4
>>@@ -3490,9 +3490,10 @@ EOF
>>  AC_LANG_RESTORE
>>
>>  # Set atomicity_dir to builtins if all but the long long test above passes.
>>-  if test "$glibcxx_cv_atomic_bool" = yes \
>>+  if ( test "$glibcxx_cv_atomic_bool" = yes \
>>     && test "$glibcxx_cv_atomic_short" = yes \
>>-     && test "$glibcxx_cv_atomic_int" = yes; then
>>+     && test "$glibcxx_cv_atomic_int" = yes ) \
>>+     || test "$atomicity_dir" = "cpu/generic/atomicity_builtins"; then
>
>Could you adjust the comment too please? Maybe something like:
>
>  # Set atomicity_dir to builtins if all but the long long test above passes,
>  # or if the builtins were already chosen (e.g. by configure.host).
>
>OK with an adjusted comment, thanks.

(And adjusted for Andreas's point about {} too of course, so we don't
start a subshell here.)




More information about the Gcc-patches mailing list