Re: [PATCH] MIPS/gas: SAA/SAAD macro clean-ups

["Pinski, Andrew" <Andrew dot Pinski at caviumnetworks dot com>]

> On Aug 21, 2014, at 5:05 AM, "Maciej W. Rozycki" <macro@codesourcery.com> wrote:
> 
> Hi,
> 
> This change removes code duplication for the SAA macro in line with other 
> such macros and also adds a !microMIPS internal consistency guard as 
> there's no microMIPS encoding of the underlying SAA/SAAD instructions.
> 
> No regressions across mips-elf, mips-sde-elf, mips-sgi-irix5,
> mips-sgi-irix6, mips-freebsd, mips-linux, mips-vxworks, mips64-freebsd,
> mips64-linux, mips64el-freebsd, mips64el-linux, mipsel-elf,
> mipsel-freebsd, mipsel-linux, mipsel-vxworks, mipsisa32-elf,
> mipsisa32-linux, mipsisa32el-elf, mipsisa32el-linux, mipsisa64-elf,
> mipsisa64-linux, mipsisa64el-elf, mipsisa64el-linux.  OK to apply?

This is ok from my point of view though I cannot approve it. 

Thanks,
Andrew

> 
> 2014-03-24  Maciej W. Rozycki  <macro@codesourcery.com>
> 
>    gas/
>    * config/tc-mips.c (macro) <M_SAA_AB>: Remove duplicate code and
>    jump to...
>    <M_SAAD_AB>: ... here.  Assert that !microMIPS.
> 
>  Maciej
> 
> binutils-gas-mips-saa.diff
> Index: binutils-fsf-trunk-quilt/gas/config/tc-mips.c
> ===================================================================
> --- binutils-fsf-trunk-quilt.orig/gas/config/tc-mips.c    2014-04-04 15:27:21.038037358 +0100
> +++ binutils-fsf-trunk-quilt/gas/config/tc-mips.c    2014-04-04 16:00:19.748876849 +0100
> @@ -11895,11 +11895,11 @@ macro (struct mips_cl_insn *ip, char *st
>    
>     case M_SAA_AB:
>       s = "saa";
> -      offbits = 0;
> -      fmt = "t,(b)";
> -      goto ld_st;
> +      goto saa_saad;
>     case M_SAAD_AB:
>       s = "saad";
> +    saa_saad:
> +      gas_assert (!mips_opts.micromips);
>       offbits = 0;
>       fmt = "t,(b)";
>       goto ld_st;