[Bug libgcj/44216] New: [4.6 regression] All libjava tests fail on IRIX 6.5: ld warns about -no-merge-exidx-entries

[Bug libgcj/44216] New: [4.6 regression] All libjava tests fail on IRIX 6.5: ld warns about -no-merge-exidx-entries

[ro at gcc dot gnu dot org]

Since the introduction of the configure check for the --no-merge-exidx-entries
linker option, all libjava tests fail on IRIX 6.5:

output is:
(null): WARNING 1  : Unknown option: -no-merge-exidx-entries (ignored).


FAIL: linking PR9577

The problem is that the SGI ld warns about unknown options as above, but still
exists with status 0.  None of the options to suppress errors/warnings has an
effect on this, neither -LD_MSG:error=1 nor -LD_MSG:off=1 nor -woff 1.

I've found no good way to handle this generically, since the AC_LINK_IFELSE
macro only logs stderr of the link command to conftest,err and the warning
goes to stdout.  I could of course simply set libgcj_cv_exidx=no for
mips-sgi-irix6.5, but perhaps there's a cleaner solution I'm just not seeing?


-- 
           Summary: [4.6 regression] All libjava tests fail on IRIX 6.5: ld
                    warns about -no-merge-exidx-entries
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: libgcj
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: ro at gcc dot gnu dot org
 GCC build triplet: mips-sgi-irix6.5
  GCC host triplet: mips-sgi-irix6.5
GCC target triplet: mips-sgi-irix6.5


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

[Bug libgcj/44216] [4.6 regression] All libjava tests fail on IRIX 6.5: ld warns about -no-merge-exidx-entries

[ro at gcc dot gnu dot org]

------- Comment #1 from ro at gcc dot gnu dot org  2010-05-20 18:23 -------
Paolo, do you happen to have an idea how to handle this cleanly?


-- 

ro at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bonzini at gnu dot org


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

[Bug libgcj/44216] [4.6 regression] All libjava tests fail on IRIX 6.5: ld warns about -no-merge-exidx-entries

[bonzini at gnu dot org]

------- Comment #2 from bonzini at gnu dot org  2010-05-21 07:27 -------
What if you call AC_LANG_WERROR just before the test?  This wouldn't be a final
patch because then you need to restore the previous value, but it's a start.

Ralf, maybe we want in Autoconf (and hence in override.m4) a mechanism to
push/pop the AC_LANG_WERROR state?


-- 

bonzini at gnu dot org changed:

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


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

[Bug libgcj/44216] [4.6 regression] All libjava tests fail on IRIX 6.5: ld warns about -no-merge-exidx-entries

[rwild at gcc dot gnu dot org]

------- Comment #3 from rwild at gcc dot gnu dot org  2010-05-21 08:15 -------
(In reply to comment #2)
> Ralf, maybe we want in Autoconf (and hence in override.m4) a mechanism to
> push/pop the AC_LANG_WERROR state?

Isn't there such a mechanism already?  If not, then there was at least a
proposed patch (series) to add such a while ago, I don't recall whether/why
that wasn't merged.


-- 


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

[Bug libgcj/44216] [4.6 regression] All libjava tests fail on IRIX 6.5: ld warns about -no-merge-exidx-entries

[ro at CeBiTec dot Uni-Bielefeld dot DE]

------- Comment #4 from ro at CeBiTec dot Uni-Bielefeld dot DE  2010-05-21 09:26 -------
Subject: Re:  [4.6 regression] All libjava tests fail on IRIX 6.5: ld warns
about -no-merge-exidx-entries

> ------- Comment #3 from rwild at gcc dot gnu dot org  2010-05-21 08:15 -------
> (In reply to comment #2)
>> Ralf, maybe we want in Autoconf (and hence in override.m4) a mechanism to
>> push/pop the AC_LANG_WERROR state?
>
> Isn't there such a mechanism already?  If not, then there was at least a
> proposed patch (series) to add such a while ago, I don't recall whether/why
> that wasn't merged.

At least not in Autoconf 2.64.  I've found a single instance in the
implementation of AC_PATH_XTRA.

I'll give it a try.

        Rainer


-- 


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

[Bug libgcj/44216] [4.6 regression] All libjava tests fail on IRIX 6.5: ld warns about -no-merge-exidx-entries

[ro at gcc dot gnu dot org]

------- Comment #5 from ro at gcc dot gnu dot org  2010-05-25 16:14 -------
Created an attachment (id=20745)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20745&action=view)
proposed patch

This patch worked for me.  It's a hack in that it does the save and restore
using lowlevel autoconf implementation details, but it gets the job done.


-- 


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

[Bug libgcj/44216] [4.6 regression] All libjava tests fail on IRIX 6.5: ld warns about -no-merge-exidx-entries

[bonzini at gnu dot org]

------- Comment #6 from bonzini at gnu dot org  2010-05-25 16:21 -------
The patch needs a fat comment saying what's going on, then it should be okay.


-- 


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

[Bug libgcj/44216] [4.6 regression] All libjava tests fail on IRIX 6.5: ld warns about -no-merge-exidx-entries

[ro at gcc dot gnu dot org]

------- Comment #7 from ro at gcc dot gnu dot org  2010-05-25 16:55 -------
Mine.


-- 

ro at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |ro at gcc dot gnu dot org
                   |dot org                     |
             Status|UNCONFIRMED                 |ASSIGNED
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2010-05-25 16:55:37
               date|                            |


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

[Bug libgcj/44216] [4.6 regression] All libjava tests fail on IRIX 6.5: ld warns about -no-merge-exidx-entries

[ro at gcc dot gnu dot org]

------- Comment #8 from ro at gcc dot gnu dot org  2010-05-25 17:00 -------
Subject: Bug 44216

Author: ro
Date: Tue May 25 16:59:45 2010
New Revision: 159833

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=159833
Log:
        PR libgcj/44216
        * configure.ac (libgcj_cv_exidx): Enable AC_LANG_WERROR.
        Save and restore werror flag.
        * configure: Regenerate.

Modified:
    trunk/libjava/ChangeLog
    trunk/libjava/configure
    trunk/libjava/configure.ac


-- 


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

[Bug libgcj/44216] [4.6 regression] All libjava tests fail on IRIX 6.5: ld warns about -no-merge-exidx-entries

[ro at gcc dot gnu dot org]

------- Comment #9 from ro at gcc dot gnu dot org  2010-05-25 17:01 -------
Fixed for 4.6.0.


-- 

ro at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|                            |http://gcc.gnu.org/ml/gcc-
                   |                            |patches/2010-
                   |                            |05/msg01889.html
             Status|ASSIGNED                    |RESOLVED
           Keywords|                            |patch
         Resolution|                            |FIXED
   Target Milestone|---                         |4.6.0


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

[Bug libgcj/44216] [4.6 regression] All libjava tests fail on IRIX 6.5: ld warns about -no-merge-exidx-entries

[ro at gcc dot gnu dot org]

------- Comment #10 from ro at gcc dot gnu dot org  2010-06-11 11:46 -------
Subject: Bug 44216

Author: ro
Date: Fri Jun 11 11:45:59 2010
New Revision: 160602

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160602
Log:
        Backport from mainline:
        2010-05-25  Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>

        PR libgcj/44216
        * configure.ac (libgcj_cv_exidx): Enable AC_LANG_WERROR.
        Save and restore werror flag.
        * configure: Regenerate.

Modified:
    branches/gcc-4_5-branch/libjava/ChangeLog
    branches/gcc-4_5-branch/libjava/configure
    branches/gcc-4_5-branch/libjava/configure.ac


-- 


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

[Bug libgcj/44216] [4.6 regression] All libjava tests fail on IRIX 6.5: ld warns about -no-merge-exidx-entries

[ro at gcc dot gnu dot org]

------- Comment #11 from ro at gcc dot gnu dot org  2010-06-11 11:49 -------
Subject: Bug 44216

Author: ro
Date: Fri Jun 11 11:49:16 2010
New Revision: 160603

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160603
Log:
        config:
        * override.m4 (AC_LANG_WERROR): Redefine to autoconf 2.62 version.

        libjava:
        Backport from mainline:
        2010-05-25  Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>

        PR libgcj/44216
        * configure.ac (libgcj_cv_exidx): Enable AC_LANG_WERROR.
        Save and restore werror flag.
        * configure: Regenerate.

Modified:
    branches/gcc-4_4-branch/config/ChangeLog
    branches/gcc-4_4-branch/config/override.m4
    branches/gcc-4_4-branch/libjava/ChangeLog
    branches/gcc-4_4-branch/libjava/configure
    branches/gcc-4_4-branch/libjava/configure.ac


-- 


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