[Patch] [MIPS] Fix Many warnings in MIPS port (Was: [PATCH] [MIPS] microMIPS gcc support)

[David Daney]

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...


> Thanks,
> 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: dd.patch
Type: text/x-patch
Size: 539 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20130413/3022ed8e/attachment.bin>