[PATCH] Add support for MIPS DSPr3
Maciej W. Rozycki
macro@imgtec.com
Tue May 10 16:36:00 GMT 2016
Hi Matthew,
> A minor revision to the MIPS DSP ASE [1] was made in conjunction with MIPSr6 in
> order to add a compact variant of the BPOSGE32 instruction. DSPr3 is only
> supported alongside MIPSr6 and is recorded in the .MIPS.abiflags ASE flags.
Thank you for your contribution. Your change looks good overall, except
for a minor nit, see below. However it adds new command-line options, so
a corresponding GAS manual update is required. Please resubmit with these
changes made.
> [1] https://imgtec.com/mips/architectures/dsp/
As a side note it looks to me like in the documentation referred this
instruction is actually missing from opcode tables, which I'm told are the
primary ISA reference, especially where there is a conflict between an
encoding stated in the relevant opcode table and one in the corresponding
individual instruction description. Can you please raise the issue with
architecture documentation maintainers?
> diff --git a/gas/testsuite/gas/mips/mips32-dspr3.s b/gas/testsuite/gas/mips/mips32-dspr3.s
> new file mode 100644
> index 0000000..4512d3a
> --- /dev/null
> +++ b/gas/testsuite/gas/mips/mips32-dspr3.s
> @@ -0,0 +1,13 @@
> +# source file to test assembly of MIPS DSP ASE Rev3 for MIPS32 instructions
> +
> + .set noreorder
> + .set noat
> +
> + .text
> +text_label:
> + bposge32c text_label
> +
> +
> +# Force at least 8 (non-delay-slot) zero bytes, to make 'objdump' print ...
> + .align 2
> + .space 8
You have inconsistent operand separation here -- please either use 1 or 2
tabs or a single space consistently (`bposge32c' can be exempted in the
single tab case as it's longer than 8 characters, but use a single space
then). I'm leaving it up to you which style to choose.
Thanks,
Maciej
More information about the Binutils
mailing list