[Bug bootstrap/9249] bootstrap fails with --enable-__cxa-atexit on Solaris

[Bug bootstrap/9249] bootstrap fails with --enable-__cxa-atexit on Solaris

[dhazeghi at yahoo dot com]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9249


dhazeghi at yahoo dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |neroden at gcc dot gnu dot
                   |                            |org


------- Additional Comments From dhazeghi at yahoo dot com  2003-10-24 23:02 -------
I can create the relevant patch for configure.texi if necessary. However, is it practical to test for 
cxa_atexit in configure, and error out if it is enabled, but not supported? Thanks.

[Bug bootstrap/9249] bootstrap fails with --enable-__cxa-atexit on Solaris

[pme at gcc dot gnu dot org]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9249



------- Additional Comments From pme at gcc dot gnu dot org  2003-10-24 23:59 -------
Yeah, a test would be good.  I should have done that after my
comment.

I /don't/ know offhand whether the --enable option affects the
default value of -fuse-cxa-exit or not; that should probably be
added to the documentation.

[Bug bootstrap/9249] bootstrap fails with --enable-__cxa-atexit on Solaris

[ebotcazou at gcc dot gnu dot org]

------- Additional Comments From ebotcazou at gcc dot gnu dot org  2003-12-01 08:57 -------
Dara, are you still volunteering to produce a patch for this PR?  I can
ultimately try to do it though.


-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |dhazeghi at yahoo dot com
   Target Milestone|---                         |3.4


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9249

[Bug bootstrap/9249] bootstrap fails with --enable-__cxa-atexit on Solaris

[dhazeghi at yahoo dot com]

------- Additional Comments From dhazeghi at yahoo dot com  2003-12-02 17:15 -------
I'm happy to do it, but can't get to it before midway next week.

-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9249

[Bug bootstrap/9249] bootstrap fails with --enable-__cxa-atexit on Solaris

[ebotcazou at gcc dot gnu dot org]

------- Additional Comments From ebotcazou at gcc dot gnu dot org  2003-12-02 18:29 -------
Fine, we're not in a hurry.  Thanks.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9249

[Bug bootstrap/9249] bootstrap fails with --enable-__cxa-atexit on Solaris

[dhazeghi at yahoo dot com]

------- Additional Comments From dhazeghi at yahoo dot com  2003-12-11 09:11 -------
How's this look (I lifted most of it directly from configure.in)?

--- doc/install.texi.old        Thu Dec 11 00:56:57 2003
+++ doc/install.texi    Thu Dec 11 01:08:53 2003
@@ -938,6 +938,14 @@
 AltiVec code when appropriate.  This option is only available for
 PowerPC systems.
 
+@item --enable-__cxa_atexit
+Define if you want to use __cxa_atexit, rather than atexit, to
+register C++ destructors for local statics and global objects.
+This is essential for fully standards-compliant handling of
+destructors, but requires __cxa_atexit in libc. This option is currently
+only available on sytems with GNU libc. When enabled, this will cause
+@option{-fuse-cxa-exit} to be passed by default.
+
 @item --enable-target-optspace
 Specify that target
 libraries should be optimized for code space instead of code speed.


Regarding a configure check for this, I haven't got any good ideas. I gues we could blacklist 
systems whose triplet didn't end in GNU, but that seems unclean. Ideas?

-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9249

[Bug bootstrap/9249] bootstrap fails with --enable-__cxa-atexit on Solaris

[cludwig at cdc dot informatik dot tu-darmstadt dot de]

------- Additional Comments From cludwig at cdc dot informatik dot tu-darmstadt dot de  2003-12-11 10:05 -------
Subject: Re:  bootstrap fails with --enable-__cxa-atexit on Solaris

On Thu, Dec 11, 2003 at 09:12:03AM -0000, dhazeghi at yahoo dot com wrote:
> 
> How's this look (I lifted most of it directly from configure.in)?
> 
> +++ doc/install.texi    Thu Dec 11 01:08:53 2003
> @@ -938,6 +938,14 @@
>  AltiVec code when appropriate.  This option is only available for
>  PowerPC systems.
>  
> +@item --enable-__cxa_atexit
> +Define if you want to use __cxa_atexit, rather than atexit, to
> +register C++ destructors for local statics and global objects.
> +This is essential for fully standards-compliant handling of
> +destructors, but requires __cxa_atexit in libc. This option is currently
> +only available on sytems with GNU libc. When enabled, this will cause
> +@option{-fuse-cxa-exit} to be passed by default.
> +
>  @item --enable-target-optspace
>  Specify that target
>  libraries should be optimized for code space instead of code speed.

It seems to state everything a user needs to know prior to calling
configure. Thank you.

> Regarding a configure check for this, I haven't got any good ideas. I gues we could blacklist 
> systems whose triplet didn't end in GNU, but that seems unclean. Ideas?

I am not familiar with the restrictions that are imposed on the tests
in  gcc's configure script. But IIUC libc has to be installed before
you build gcc anyway. Then why not try to link a program that
explicitly references __cxa_atexit. If the linker or the program
fails, then libc does not provide __cxx_atexit. I have something in
mind along the line of:

#### begin of cxa_atexit_test.c ####

void __cxa_atexit();

int main() {
  typedef void (*ExitFunctionPtr)();

  ExitFunctionPtr cxa_atexit_ptr = __cxa_atexit;

  return cxa_atexit_ptr ? return 0 : return 1;
}

#### end of cxa_atexit_test.c ####

(The program runs on linux / gcc 3.3.2 but fails to link on Solaris 8
/ gcc 2.95, as expected. I didn't perform more thorough tests.)
If the gcc build process only assumes a K&R C-compiler then the
program probably needs to be adapted. I am much more used to C++
whence I'd need to look up the K&R syntax.

Regards

Christoph


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9249

[Bug bootstrap/9249] bootstrap fails with --enable-__cxa-atexit on Solaris

[ebotcazou at gcc dot gnu dot org]

------- Additional Comments From ebotcazou at gcc dot gnu dot org  2003-12-11 18:29 -------
I personally find that the blurb in the docs is sufficient, but a configure
check could be indeed helpful.  The AC_CHECK_FUNC autoconf macro appears to be
the simplest method.



-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9249

[Bug bootstrap/9249] bootstrap fails with --enable-__cxa-atexit on Solaris

[dhazeghi at yahoo dot com]

------- Additional Comments From dhazeghi at yahoo dot com  2003-12-11 20:42 -------
Okay, tests fine on Solaris/Darwin/i686-Linux. I'll submit this in a day or two, unless there are 
objections.

-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9249

[Bug bootstrap/9249] bootstrap fails with --enable-__cxa-atexit on Solaris

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2003-12-20 08:34 -------
Dara has a patch assigning it to him so that people know that he is working on it.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |dhazeghi at yahoo dot com
                   |dot org                     |
             Status|NEW                         |ASSIGNED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9249

[Bug bootstrap/9249] bootstrap fails with --enable-__cxa-atexit on Solaris

[neroden at gcc dot gnu dot org]

------- Additional Comments From neroden at gcc dot gnu dot org  2004-01-10 01:36 -------
Status of the doc patch?  Has it been committed yet?

-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9249

[Bug bootstrap/9249] bootstrap fails with --enable-__cxa-atexit on Solaris

[steven at gcc dot gnu dot org]

------- Additional Comments From steven at gcc dot gnu dot org  2004-01-10 22:40 -------
Enhancement, so moving target milestone.  The patch is three months old 
and apparently never reviewed.  Perhaps someone is gung-ho on SPARC and 
still feels like reviewing/approving/commiting it.  If not, see you 
next release. 
 

-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9249

[Bug bootstrap/9249] bootstrap fails with --enable-__cxa-atexit on Solaris

[steven at gcc dot gnu dot org]

------- Additional Comments From steven at gcc dot gnu dot org  2004-01-10 22:42 -------
actually moved this time. 

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|3.4.0                       |3.5.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9249

[Bug bootstrap/9249] bootstrap fails with --enable-__cxa-atexit on Solaris

[ebotcazou at gcc dot gnu dot org]

------- Additional Comments From ebotcazou at gcc dot gnu dot org  2004-01-11 06:47 -------
> The patch is three months old and apparently never reviewed.

Uh? Please look more carefully. And I'd like to have it in the 3.4 release.

Nathanael, would you mind to approve it?


-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|3.5.0                       |3.4.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9249

[Bug bootstrap/9249] bootstrap fails with --enable-__cxa-atexit on Solaris

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2004-01-14 03:49 -------
No work and this is not a regression so moving to 3.4.1.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|3.4.0                       |3.4.1


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9249

[Bug bootstrap/9249] bootstrap fails with --enable-__cxa-atexit on Solaris

[ebotcazou at gcc dot gnu dot org]

------- Additional Comments From ebotcazou at gcc dot gnu dot org  2004-01-14 05:28 -------
Think about it two seconds: how a problem that is not a regression can be
targeted to a .1 release ???

We have a patch (which looks ok to me), it has been successfully tested, we only
need an approval.

Nathanael, are you around?


-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|3.4.1                       |3.4.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9249

[Bug bootstrap/9249] bootstrap fails with --enable-__cxa-atexit on Solaris

[dhazeghi at yahoo dot com]

------- Additional Comments From dhazeghi at yahoo dot com  2004-01-16 04:06 -------
Oops, looks like I forgot to send this off to the list. Okay, I'll retest, and barring disaster, send it to 
the list...

-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9249

[Bug bootstrap/9249] bootstrap fails with --enable-__cxa-atexit on Solaris

[dhazeghi at yahoo dot com]

------- Additional Comments From dhazeghi at yahoo dot com  2004-01-17 08:00 -------
Patch submitted: <http://gcc.gnu.org/ml/gcc-patches/2004-01/msg01658.html> and <http://
gcc.gnu.org/ml/gcc-patches/2004-01/msg01664.html>. Just waiting for a review now...

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |patch


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9249

[Bug bootstrap/9249] bootstrap fails with --enable-__cxa-atexit on Solaris

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2004-01-31 04:12 -------
Subject: Bug 9249

CVSROOT:	/cvs/gcc
Module name:	gcc
Branch: 	gcc-3_4-branch
Changes by:	pinskia@gcc.gnu.org	2004-01-31 04:12:22

Modified files:
	gcc            : ChangeLog configure configure.ac 

Log message:
	2004-01-30  Dara Hazeghi  <dhazeghi@yahoo.com>
	
	PR bootstrap/9249
	* configure.ac: Disable __cxa_atexit if not supported.
	* configure: Regenerate.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=2.2326.2.118&r2=2.2326.2.119
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/configure.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.794.2.5&r2=1.794.2.6
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/configure.ac.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=2.6.2.5&r2=2.6.2.6



-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9249

[Bug bootstrap/9249] bootstrap fails with --enable-__cxa-atexit on Solaris

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2004-01-31 04:15 -------
Subject: Bug 9249

CVSROOT:	/cvs/gcc
Module name:	gcc
Branch: 	gcc-3_4-branch
Changes by:	pinskia@gcc.gnu.org	2004-01-31 04:14:57

Modified files:
	gcc            : ChangeLog 
	gcc/doc        : install.texi 

Log message:
	2004-01-30  Dara Hazeghi  <dhazeghi@yahoo.com>
	
	PR bootstrap/9249
	* doc/install.texi: document --enable-__cxa_atexit option.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=2.2326.2.119&r2=2.2326.2.120
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/doc/install.texi.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.248.4.3&r2=1.248.4.4



-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9249

[Bug bootstrap/9249] bootstrap fails with --enable-__cxa-atexit on Solaris

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2004-01-31 04:18 -------
Subject: Bug 9249

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	pinskia@gcc.gnu.org	2004-01-31 04:18:16

Modified files:
	gcc            : ChangeLog configure configure.ac 
	gcc/doc        : install.texi 

Log message:
	2004-01-30  Dara Hazeghi  <dhazeghi@yahoo.com>
	
	PR bootstrap/9249
	* doc/install.texi: document --enable-__cxa_atexit option.
	* configure.ac: Disable __cxa_atexit if not supported.
	* configure: Regenerate.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&r1=2.2575&r2=2.2576
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/configure.diff?cvsroot=gcc&r1=1.798&r2=1.799
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/configure.ac.diff?cvsroot=gcc&r1=2.10&r2=2.11
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/doc/install.texi.diff?cvsroot=gcc&r1=1.251&r2=1.252



-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9249

[Bug bootstrap/9249] bootstrap fails with --enable-__cxa-atexit on Solaris

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2004-01-31 04:19 -------
Fixed for 3.4.0 and 3.5.0.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9249