target.h (targetm.cxx.use_atexit_for_cxa_atexit): New target hook.

	* target.h (targetm.cxx.use_atexit_for_cxa_atexit): New target
	hook.
	* target-def.h: (TARGET_CXX_USE_ATEXIT_FOR_CXA_ATEXIT): Define
	default.
	* config/i386/mingw32.h (TARGET_CXX_USE_ATEXIT_FOR_CXA_ATEXIT):
	Override default.
	* doc/tm.texi (TARGET_CXX_USE_ATEXIT_FOR_CXA_ATEXIT): Document.
	* configure.ac (use_cxa_atexit): As a special case, don't test
	for libc definition of __cxa_atexit on mingw32
	* configure: Regenerate.
	* config.gcc (i[34567]86-pc-mingw32): Default to
	enable__cxa_atexit=yes.

cp

	* decl.c (get_atexit_node): Reference atexit, not __cxa_exit.
	if targetm.cxx.use_atexit_for cxa_atexit.
	(start_cleanup_fn): Likewise.
	(register_dtor_fn): Likewise.

From-SVN: r118371

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index ab2d0d1e94627774fd7442b891371248e285b765..6332e06d05c597585dc48f1ccc95d84a3ef9139c 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,18 @@
+2006-11-01  Danny Smith  <dannysmith@users.sourceforge.net>
+
+       * target.h (targetm.cxx.use_atexit_for_cxa_atexit): New target
+       hook.
+       * target-def.h: (TARGET_CXX_USE_ATEXIT_FOR_CXA_ATEXIT): Define
+       default.
+       * config/i386/mingw32.h (TARGET_CXX_USE_ATEXIT_FOR_CXA_ATEXIT):
+       Override default.
+       * doc/tm.texi (TARGET_CXX_USE_ATEXIT_FOR_CXA_ATEXIT): Document.
+       * configure.ac (use_cxa_atexit): As a special case, don't test
+       for libc definition of __cxa_atexit on mingw32
+       * configure: Regenerate.
+       * config.gcc (i[34567]86-pc-mingw32): Default to
+       enable__cxa_atexit=yes.
+
 2006-11-01  Kaveh R. Ghazi  <ghazi@caip.rutgers.edu>
 
        * builtins.def (gamma, lgamma): Use ATTR_MATHFN_FPROUNDING_STORE.

diff --git a/gcc/config.gcc b/gcc/config.gcc
index bbe82f069c9b39f8355fa8c91e3e57a76d66d919..9ab3a659c6cb5a65e0569d417090b0e1658681bf 100644 (file)
--- a/gcc/config.gcc
+++ b/gcc/config.gcc
@@ -1310,6 +1310,7 @@ i[34567]86-*-mingw32*)
        extra_options="${extra_options} i386/cygming.opt"
        extra_objs="winnt.o winnt-stubs.o"
        cxx_target_objs=winnt-cxx.o
+       default_use_cxa_atexit=yes
        case ${enable_threads} in
          "" | yes | win32)
          thread_file='win32'

diff --git a/gcc/config/i386/mingw32.h b/gcc/config/i386/mingw32.h
index 6fd5d94cfd51642140e3c0db60004bef69df910f..28c2fb9fbb8e312d4f6d5d91cf768e1d31fe9b01 100644 (file)
--- a/gcc/config/i386/mingw32.h
+++ b/gcc/config/i386/mingw32.h
@@ -112,3 +112,7 @@ do {                                                         \
 /* mingw32 uses the  -mthreads option to enable thread support.  */
 #undef GOMP_SELF_SPECS
 #define GOMP_SELF_SPECS "%{fopenmp: -mthreads}"
+
+/* mingw32 atexit function is safe to use in shared libraries.  Use it
+   to register C++ static destructors.  */
+#define TARGET_CXX_USE_ATEXIT_FOR_CXA_ATEXIT hook_bool_void_true

diff --git a/gcc/configure b/gcc/configure
index cbf94b59f806613e3d5bb532acee78c767ccbb69..49535d590f7fd69fbef8561620fe46d4156f420b 100755 (executable)
--- a/gcc/configure
+++ b/gcc/configure
@@ -12428,7 +12428,14 @@ use_cxa_atexit=no
 if test x$enable___cxa_atexit = xyes || \
    test x$enable___cxa_atexit = x -a x$default_use_cxa_atexit = xyes; then
   if test x$host = x$target; then
-    echo "$as_me:$LINENO: checking for __cxa_atexit" >&5
+    case $host in
+      # mingw32 doesn't have __cxa_atexit but uses atexit registration
+      # keyed to flag_use_cxa_atexit
+      *-*-mingw32*)
+       use_cxa_atexit=yes
+       ;;
+      *)
+       echo "$as_me:$LINENO: checking for __cxa_atexit" >&5
 echo $ECHO_N "checking for __cxa_atexit... $ECHO_C" >&6
 if test "${ac_cv_func___cxa_atexit+set}" = set; then
   echo $ECHO_N "(cached) $ECHO_C" >&6
@@ -12524,6 +12531,8 @@ else
   echo "__cxa_atexit can't be enabled on this target"
 fi
 
+       ::
+    esac
   else
     # We can't check for __cxa_atexit when building a cross, so assume
     # it is available

diff --git a/gcc/configure.ac b/gcc/configure.ac
index 0b42531ff6d5f751279231bb5e7a95456fa586e2..7b571385ce4df9f11cadefbabe6610931e17555c 100644 (file)
--- a/gcc/configure.ac
+++ b/gcc/configure.ac
@@ -1429,8 +1429,17 @@ use_cxa_atexit=no
 if test x$enable___cxa_atexit = xyes || \
    test x$enable___cxa_atexit = x -a x$default_use_cxa_atexit = xyes; then
   if test x$host = x$target; then
-    AC_CHECK_FUNC(__cxa_atexit,[use_cxa_atexit=yes],
-       [echo "__cxa_atexit can't be enabled on this target"])
+    case $host in
+      # mingw32 doesn't have __cxa_atexit but uses atexit registration
+      # keyed to flag_use_cxa_atexit
+      *-*-mingw32*)
+       use_cxa_atexit=yes
+       ;;
+      *)
+       AC_CHECK_FUNC(__cxa_atexit,[use_cxa_atexit=yes],
+         [echo "__cxa_atexit can't be enabled on this target"])
+       ::
+    esac
   else
     # We can't check for __cxa_atexit when building a cross, so assume
     # it is available 

diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index add6b2c33898fbe5e425ffdd8b7de305fdceb2bc..6e30ada48737594d57ceed29243d810996cbc6e1 100644 (file)
--- a/gcc/cp/ChangeLog
+++ b/gcc/cp/ChangeLog
@@ -1,3 +1,10 @@
+2006-11-01  Danny Smith  <dannysmith@users.sourceforge.net>
+
+       * decl.c (get_atexit_node): Reference atexit, not __cxa_exit.
+       if targetm.cxx.use_atexit_for cxa_atexit.
+       (start_cleanup_fn): Likewise.
+       (register_dtor_fn): Likewise.
+
 2006-09-25  Geoffrey Keating  <geoffk@apple.com>
 
        * decl2.c (cp_write_global_declarations): Rename from

diff --git a/gcc/cp/decl.c b/gcc/cp/decl.c
index 66c34e418da9f24bc03864b6b0e93d6c171e5ee0..cc74908c152c27acf073fa95454902e46b0d5edb 100644 (file)
--- a/gcc/cp/decl.c
+++ b/gcc/cp/decl.c
@@ -5348,7 +5348,7 @@ get_atexit_node (void)
   if (atexit_node)
     return atexit_node;
 
-  if (flag_use_cxa_atexit)
+  if (flag_use_cxa_atexit && !targetm.cxx.use_atexit_for_cxa_atexit ())
     {
       /* The declaration for `__cxa_atexit' is:
 
@@ -5436,6 +5436,8 @@ start_cleanup_fn (void)
   tree parmtypes;
   tree fntype;
   tree fndecl;
+  bool use_cxa_atexit = flag_use_cxa_atexit
+                       && !targetm.cxx.use_atexit_for_cxa_atexit ();
 
   push_to_top_level ();
 
@@ -5448,7 +5450,7 @@ start_cleanup_fn (void)
      We'll just ignore it.  After we implement the new calling
      convention for destructors, we can eliminate the use of
      additional cleanup functions entirely in the -fnew-abi case.  */
-  if (flag_use_cxa_atexit)
+  if (use_cxa_atexit)
     parmtypes = tree_cons (NULL_TREE, ptr_type_node, parmtypes);
   /* Build the function type itself.  */
   fntype = build_function_type (void_type_node, parmtypes);
@@ -5468,7 +5470,7 @@ start_cleanup_fn (void)
   DECL_DECLARED_INLINE_P (fndecl) = 1;
   DECL_INTERFACE_KNOWN (fndecl) = 1;
   /* Build the parameter.  */
-  if (flag_use_cxa_atexit)
+  if (use_cxa_atexit)
     {
       tree parmdecl;
 
@@ -5537,7 +5539,7 @@ register_dtor_fn (tree decl)
   cxx_mark_addressable (cleanup);
   mark_used (cleanup);
   cleanup = build_unary_op (ADDR_EXPR, cleanup, 0);
-  if (flag_use_cxa_atexit)
+  if (flag_use_cxa_atexit && !targetm.cxx.use_atexit_for_cxa_atexit ())
     {
       args = tree_cons (NULL_TREE,
                        build_unary_op (ADDR_EXPR, get_dso_handle_node (), 0),

diff --git a/gcc/doc/tm.texi b/gcc/doc/tm.texi
index cc448bcc687a05b344b487549fe92945e372f043..c4dc052ddbe3f3fcc0969b6b5dc487d26784f55f 100644 (file)
--- a/gcc/doc/tm.texi
+++ b/gcc/doc/tm.texi
@@ -8971,6 +8971,14 @@ should be used to register static destructors when @option{-fuse-cxa-atexit}
 is in effect.  The default is to return false to use @code{__cxa_atexit}.
 @end deftypefn
 
+@deftypefn {Target Hook} bool TARGET_CXX_USE_ATEXIT_FOR_CXA_ATEXIT (void)
+This hook returns true if the target @code{atexit} function can be used
+in the same manner as @code{__cxa_atexit} to register C++ static
+destructors. This requires that @code{atexit}-registered functions in
+shared libraries are run in the correct order when the libraries are
+unloaded. The default is to return false.
+@end deftypefn
+
 @deftypefn {Target Hook} void TARGET_CXX_ADJUST_CLASS_AT_DEFINITION (tree @var{type})
 @var{type} is a C++ class (i.e., RECORD_TYPE or UNION_TYPE) that has just been
 defined.  Use this hook to make adjustments to the class (eg, tweak

diff --git a/gcc/target-def.h b/gcc/target-def.h
index 74cf86583d74f7ddb4f6d4cf8520d8599631cbe6..82e711a79c7677f6adaa901ed1cbb2cf381ebeae 100644 (file)
--- a/gcc/target-def.h
+++ b/gcc/target-def.h
@@ -583,6 +583,10 @@ Foundation, 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
 #define TARGET_CXX_USE_AEABI_ATEXIT hook_bool_void_false
 #endif
 
+#ifndef TARGET_CXX_USE_ATEXIT_FOR_CXA_ATEXIT
+#define TARGET_CXX_USE_ATEXIT_FOR_CXA_ATEXIT hook_bool_void_false
+#endif
+
 #ifndef TARGET_CXX_ADJUST_CLASS_AT_DEFINITION
 #define TARGET_CXX_ADJUST_CLASS_AT_DEFINITION hook_void_tree
 #endif
@@ -599,6 +603,7 @@ Foundation, 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
     TARGET_CXX_DETERMINE_CLASS_DATA_VISIBILITY,        \
     TARGET_CXX_CLASS_DATA_ALWAYS_COMDAT,        \
     TARGET_CXX_USE_AEABI_ATEXIT,               \
+    TARGET_CXX_USE_ATEXIT_FOR_CXA_ATEXIT,      \
     TARGET_CXX_ADJUST_CLASS_AT_DEFINITION      \
   }
 

diff --git a/gcc/target.h b/gcc/target.h
index c18fd62b0e64d6abf5e2fdcf31b9d89a48f3dcd6..b193b624f39ee43488a221f500fe8b2ddbd040fb 100644 (file)
--- a/gcc/target.h
+++ b/gcc/target.h
@@ -776,6 +776,9 @@ struct gcc_target
     /* Returns true if __aeabi_atexit should be used to register static
        destructors.  */
     bool (*use_aeabi_atexit) (void);
+    /* Returns true if target may use atexit in the same manner as
+    __cxa_atexit  to register static destructors.  */
+    bool (*use_atexit_for_cxa_atexit) (void);
     /* TYPE is a C++ class (i.e., RECORD_TYPE or UNION_TYPE) that
        has just been defined.  Use this hook to make adjustments to the
        class  (eg, tweak visibility or perform any other required