GNU bug report logs - #16452
opt_duplicate_compiler_generated_deps is harmful on Solaris

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: "Gary V. Vaughan" <gary <at> gnu.org>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#16452: closed (opt_duplicate_compiler_generated_deps is
 harmful on Solaris)
Date: Fri, 17 Jan 2014 21:19:02 +0000

[Message part 1 (text/plain, inline)]

Your message dated Sat, 18 Jan 2014 10:18:39 +1300
with message-id <5ACD9FF7-973F-45F7-B178-476D2131F767 <at> gnu.org>
and subject line Re: bug#16452: opt_duplicate_compiler_generated_deps is harmful on Solaris
has caused the debbugs.gnu.org bug report #16452,
regarding opt_duplicate_compiler_generated_deps is harmful on Solaris
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
16452: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=16452
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Rainer Orth <ro <at> CeBiTec.Uni-Bielefeld.DE>
To: bug-libtool <at> gnu.org
Subject: opt_duplicate_compiler_generated_deps is harmful on Solaris
Date: Wed, 15 Jan 2014 13:59:11 +0100

As described in GCC bug PR target/59788

	Mixing libc and libgcc_s unwinders on 64-bit Solaris 10+/x86 breaks EH
        http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59788

linking e.g. C++ code with -Wl,-Bdirect on 64-bit Solaris 10+/x86 causes
unwinding failures caused by mixing the libc and libgcc_s unwinders in a
single executable.

I've fixed this by a patch to cause every shared object to be linked
with a mapfile to force direct binding to the libgcc_s unwinder,
avoiding the harmful mixing.  Unfortunately, for this change to work
requires to disable the opt_duplicate_compiler_generated_deps
optimization (included in that patch for the copy of libtool in the gcc
tree):

diff --git a/ltmain.sh b/ltmain.sh
--- a/ltmain.sh
+++ b/ltmain.sh
@@ -976,7 +976,7 @@ func_enable_tag ()
 
 
   case $host in
-    *cygwin* | *mingw* | *pw32* | *cegcc*)
+    *cygwin* | *mingw* | *pw32* | *cegcc* | *solaris2* )
       # don't eliminate duplications in $postdeps and $predeps
       opt_duplicate_compiler_generated_deps=:
       ;;

I think that patch should go into upstream libtool as well, although
this won't help for all the copies of libtool already included in
released packages.  As described in the PR, I'm uncertain how to move
forward here.  Suggestions?

Thanks.
        Rainer

-- 
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University

[Message part 3 (message/rfc822, inline)]

From: "Gary V. Vaughan" <gary <at> gnu.org>
To: Rainer Orth <ro <at> CeBiTec.Uni-Bielefeld.DE>
Cc: 16452-done <at> debbugs.gnu.org, Libtool Patches List <libtool-patches <at> gnu.org>
Subject: Re: bug#16452: opt_duplicate_compiler_generated_deps is harmful on
 Solaris
Date: Sat, 18 Jan 2014 10:18:39 +1300

Hi Rainer,

On Jan 16, 2014, at 1:59 AM, Rainer Orth <ro <at> CeBiTec.Uni-Bielefeld.DE> wrote:
> diff --git a/ltmain.sh b/ltmain.sh
> --- a/ltmain.sh
> +++ b/ltmain.sh
> @@ -976,7 +976,7 @@ func_enable_tag ()
> 
> 
>   case $host in
> -    *cygwin* | *mingw* | *pw32* | *cegcc*)
> +    *cygwin* | *mingw* | *pw32* | *cegcc* | *solaris2* )
>       # don't eliminate duplications in $postdeps and $predeps
>       opt_duplicate_compiler_generated_deps=:
>       ;;
> 
> I think that patch should go into upstream libtool as well, although
> this won't help for all the copies of libtool already included in
> released packages.  As described in the PR, I'm uncertain how to move
> forward here.  Suggestions?

I took the liberty of writing a ChangeLog on your behalf.

Pushed, this will be in the incoming release as long as testing doesn't find
any issues.  Thanks.

Cheers,
-- 
Gary V. Vaughan (gary AT gnu DOT org)

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.