[patch] --enable-dynamic-string default for mingw-w64
Kai Tietz
ktietz70@googlemail.com
Tue Sep 20 08:36:00 GMT 2011
2011/9/20 JonY <jon_y@users.sourceforge.net>:
> Hi,
>
> Its been used in the automated toolchain builds for sometime, seems like
> a good idea to enable it by default. It can be easily changed to match
> for all mingw as well if needed.
>
> OK for trunk?
>
> Index: libstdc++-v3/configure.ac
> ===================================================================
> --- libstdc++-v3/configure.ac (revision 178946)
> +++ libstdc++-v3/configure.ac (working copy)
> @@ -132,7 +132,15 @@
> GLIBCXX_ENABLE_DEBUG([no])
> GLIBCXX_ENABLE_PARALLEL([yes])
> GLIBCXX_ENABLE_CXX_FLAGS
> -GLIBCXX_ENABLE_FULLY_DYNAMIC_STRING([no])
> +case "$host" in
> + # Enable fully ynamic strings on mingw-w64 by default
> + i?86-w64-mingw* | x86_64-w64-mingw*)
> + GLIBCXX_ENABLE_FULLY_DYNAMIC_STRING([yes])
> + ;;
> + *)
> + GLIBCXX_ENABLE_FULLY_DYNAMIC_STRING([no])
> + ;;
> +esac
> GLIBCXX_ENABLE_EXTERN_TEMPLATE([yes])
>
> # Checks for operating systems support that doesn't require linking.
This patch looks reasonable to me. Note that this is no approval for
it, as I'm not a libstdc++ maintainer. But I can confirm that patch
is ok from perspective of windows-target maintainer.
Just one question I have here. Why do we enable this option only for
w64 flavor? AFAIK is mingw.org also building their libstdc++ with
enabling this configure-option
Kai
More information about the Gcc-patches
mailing list