On Fri, May 20, 2016 at 09:01:33AM +0100, Maciej W. Rozycki wrote: > On Wed, 18 May 2016, James Hogan wrote: > > > The versions of the __write_{32,64}bit_gc0_register() macros for when > > there is no virt support in the assembler use the "J" inline asm > > constraint to allow integer zero, but this needs to be accompanied by > > the "z" formatting string so that it turns into $0. Fix both macros to > > do this. > > NB `z' here is an "operand code" in GCC-speak. There's a list of the > MIPS-specific ones in gcc/config/mips/mips.c above `mips_print_operand'. > There are a few generic operand codes as well, most notably `a' to print > an address, matching the `p' constraint. I think it would be good to have > this all documented in the GCC manual sometime. Thanks Maciej! To be honest I pretty much guessed at what they were called. Good to see 'z' does do what I thought. I couldn't find any documentation online for the MIPS specific operand codes, so having them documented with GCC would be an excellent idea! > > > Fixes: bad50d79255a ("MIPS: Fix VZ probe gas errors with binutils <2.24") > > Signed-off-by: James Hogan <james.hogan@xxxxxxxxxx> > > Cc: Ralf Baechle <ralf@xxxxxxxxxxxxxx> > > Cc: linux-mips@xxxxxxxxxxxxxx > > --- > > Reviewed-by: Maciej W. Rozycki <macro@xxxxxxxxxx> Thanks! James
Attachment:
signature.asc
Description: Digital signature