This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
RE: [Patch] [MIPS] Fix Many warnings in MIPS port (Was: [PATCH] [MIPS] microMIPS gcc support)
- From: "Moore, Catherine" <Catherine_Moore at mentor dot com>
- To: David Daney <ddaney dot cavm at gmail dot com>
- Cc: "Rozycki, Maciej" <Maciej_Rozycki at mentor dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Richard Sandiford <rdsandiford at googlemail dot com>, "Moore, Catherine" <Catherine_Moore at mentor dot com>
- Date: Sun, 14 Apr 2013 20:27:07 +0000
- Subject: RE: [Patch] [MIPS] Fix Many warnings in MIPS port (Was: [PATCH] [MIPS] microMIPS gcc support)
- References: <FD3DCEAC5B03E9408544A1E416F11242F8F97F9E at NA-MBX-04 dot mgc dot mentorg dot com> <516898BB dot 5050903 at gmail dot com>
> -----Original Message-----
> From: David Daney [mailto:ddaney.cavm@gmail.com]
> Sent: Friday, April 12, 2013 7:29 PM
> To: Moore, Catherine
> Cc: Rozycki, Maciej; gcc-patches@gcc.gnu.org; Richard Sandiford
> Subject: Re: [Patch] [MIPS] Fix Many warnings in MIPS port (Was: [PATCH]
> [MIPS] microMIPS gcc support)
>
> On 04/12/2013 03:07 PM, Moore, Catherine wrote:
> > Hi David,
> > Please try the attached patch. Is this OK to checkin?
>
> I don't think it is correct...
And you would be right. I attached the wrong patch. Try this one instead:
Index: configure
===================================================================
--- configure (revision 197950)
+++ configure (working copy)
@@ -17830,7 +17830,7 @@
lt_dlunknown=0; lt_dlno_uscore=1; lt_dlneed_uscore=2
lt_status=$lt_dlunknown
cat > conftest.$ac_ext <<_LT_EOF
-#line 17831 "configure"
+#line 17833 "configure"
#include "confdefs.h"
#if HAVE_DLFCN_H
@@ -17936,7 +17936,7 @@
lt_dlunknown=0; lt_dlno_uscore=1; lt_dlneed_uscore=2
lt_status=$lt_dlunknown
cat > conftest.$ac_ext <<_LT_EOF
-#line 17937 "configure"
+#line 17939 "configure"
#include "confdefs.h"
#if HAVE_DLFCN_H
@@ -25766,7 +25766,7 @@
gcc_cv_as_micromips_support=no
if test x$gcc_cv_as != x; then
$as_echo '.set micromips' > conftest.s
- if { ac_try='$gcc_cv_as $gcc_cv_as_flags -o conftest.o conftest.s >&5'
+ if { ac_try='$gcc_cv_as $gcc_cv_as_flags --fatal-warnings -o conftest.o conftest.s >&5'
{ { eval echo "\"\$as_me\":${as_lineno-$LINENO}: \"$ac_try\""; } >&5
(eval $ac_try) 2>&5
ac_status=$?
Index: configure.ac
===================================================================
--- configure.ac (revision 197950)
+++ configure.ac (working copy)
@@ -4058,7 +4058,7 @@
[Define if your assembler supports .gnu_attribute.])])
gcc_GAS_CHECK_FEATURE([.micromips support],
- gcc_cv_as_micromips_support,,,
+ gcc_cv_as_micromips_support,,[--fatal-warnings],
[.set micromips],,
[AC_DEFINE(HAVE_GAS_MICROMIPS, 1,
[Define if your assembler supports the .set micromips directive])])
Sorry for the confusion.
Catherine
> >
> > 2013-04-12 Catherine Moore<clm@codesourcery.com>
> >
> > * configure.ac (.micromips support): Add --fatal-warnings option.
> > * configure: Regenerate.
> >
> >
> [...]
> >
> > Index: configure.ac
> >
> ==========================================================
> =========
> > --- configure.ac (revision 197936)
> > +++ configure.ac (working copy)
> > @@ -4051,6 +4051,12 @@ LCF0:
> > [AC_DEFINE(HAVE_AS_NO_SHARED, 1,
> > [Define if the assembler understands -mno-shared.])])
> >
> > + gcc_GAS_CHECK_FEATURE([.micromips support],
> > + gcc_cv_as_micromips_support,,[--fatal-warnings],
> > + [.set micromips],,
> > + [AC_DEFINE(HAVE_GAS_MICROMIPS, 1,
> > + [Define if your assembler supports the .set micromips
> > + directive])])
> > +
>
> There is already an existing check. Just modify that one instead of adding a
> duplicate. Something like the attached (untested)
>
> > gcc_GAS_CHECK_FEATURE([.gnu_attribute support],
> > gcc_cv_as_mips_gnu_attribute, [2,18,0],,
> > [.gnu_attribute 4,1],,
>
>
> I didn't have time to test this yet. I may try Monday.
>
> David Daney