[Ping] Port of VTV for Cygwin and MinGW

Jonathan Wakely jwakely@redhat.com
Tue Sep 23 10:22:00 GMT 2014


On 19/09/14 00:23 +0200, Patrick Wollgast wrote:
>Index: libstdc++-v3/acinclude.m4
>===================================================================
>--- libstdc++-v3/acinclude.m4	(Revision 214408)
>+++ libstdc++-v3/acinclude.m4	(Arbeitskopie)
>@@ -2321,7 +2321,17 @@ AC_DEFUN([GLIBCXX_ENABLE_VTABLE_VERIFY],
>   AC_MSG_RESULT([$enable_vtable_verify])
> 
>   if test $enable_vtable_verify = yes; then
>+    case ${target_os} in
>+  cygwin*|mingw32*)
>+    VTV_CXXFLAGS="-fvtable-verify=std -Wl,-lvtv,-u_vtable_map_vars_start,-u_vtable_map_vars_end"
>+    vtv_cygmin="yes"
>+    ;;
>+  *)
>     VTV_CXXFLAGS="-fvtable-verify=std -Wl,-u_vtable_map_vars_start,-u_vtable_map_vars_end"
>+    vtv_cygmin="no"
>+    ;;
>+    esac
>+    AM_CONDITIONAL(VTV_CYGMIN, test $vtv_cygmin = yes)
>     VTV_PCH_CXXFLAGS="-fvtable-verify=std"
>     VTV_CXXLINKFLAGS="-L${toplevel_builddir}/libvtv/.libs -Wl,--rpath -Wl,${toplevel_builddir}/libvtv/.libs"		
>   else

The lines between case and esac should be indented by four more
spaces.

Apart from that the libstdc++ changes are OK and I'll approve them.



More information about the Gcc-patches mailing list