public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] MIPS/opcodes: Fix ALNV.PS's no-delay-slot annotation
@ 2011-10-24 21:15 Maciej W. Rozycki
  2011-11-15 17:30 ` Richard Sandiford
  0 siblings, 1 reply; 2+ messages in thread
From: Maciej W. Rozycki @ 2011-10-24 21:15 UTC (permalink / raw)
  To: binutils; +Cc: Richard Sandiford

Hi,

 The switch from TRAP to NODS was missed for "alnv.ps" due to patch 
reordering.  No regressions from the change below for mips-linux-gnu or 
mips-sde-elf.  OK to apply?

2011-10-24  Maciej W. Rozycki  <macro@codesourcery.com>

	opcodes/
	* micromips-opc.c (micromips_opcodes): Use NODS rather than TRAP 
	for "alnv.ps".

  Maciej

binutils-opcodes-alnv-ps-fix.diff
Index: binutils-fsf-trunk-quilt/opcodes/micromips-opc.c
===================================================================
--- binutils-fsf-trunk-quilt.orig/opcodes/micromips-opc.c	2011-10-24 14:49:18.505931218 +0100
+++ binutils-fsf-trunk-quilt/opcodes/micromips-opc.c	2011-10-24 14:49:20.335887136 +0100
@@ -175,9 +175,9 @@ const struct mips_opcode micromips_opcod
 {"addu",    "md,me,ml",	    0x0400,     0xfc01,	0,			WR_md|RD_me|RD_ml,	I1	},
 {"addu",    "d,v,t",	0x00000150, 0xfc0007ff,	WR_d|RD_s|RD_t,		0,		I1	},
 {"addu",    "t,r,I",	0,    (int) M_ADDU_I,	INSN_MACRO,		0,		I1	},
-/* We have no flag to mark the read from "y", so we use TRAP to disable
+/* We have no flag to mark the read from "y", so we use NODS to disable
    delay slot scheduling of ALNV.PS altogether.  */
-{"alnv.ps", "D,V,T,y",	0x54000019, 0xfc00003f,	TRAP|WR_D|RD_S|RD_T|FP_D, 0,		I1	},
+{"alnv.ps", "D,V,T,y",	0x54000019, 0xfc00003f,	NODS|WR_D|RD_S|RD_T|FP_D, 0,		I1	},
 {"and",     "mf,mt,mg",	    0x4480,     0xffc0,	0,			WR_mf|RD_mf|RD_mg,	I1	},
 {"and",     "mf,mg,mx",	    0x4480,     0xffc0,	0,			WR_mf|RD_mf|RD_mg,	I1	},
 {"and",     "d,v,t",	0x00000250, 0xfc0007ff,	WR_d|RD_s|RD_t,		0,		I1	},

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] MIPS/opcodes: Fix ALNV.PS's no-delay-slot annotation
  2011-10-24 21:15 [PATCH] MIPS/opcodes: Fix ALNV.PS's no-delay-slot annotation Maciej W. Rozycki
@ 2011-11-15 17:30 ` Richard Sandiford
  0 siblings, 0 replies; 2+ messages in thread
From: Richard Sandiford @ 2011-11-15 17:30 UTC (permalink / raw)
  To: Maciej W. Rozycki; +Cc: binutils

"Maciej W. Rozycki" <macro@codesourcery.com> writes:
> 2011-10-24  Maciej W. Rozycki  <macro@codesourcery.com>
>
> 	opcodes/
> 	* micromips-opc.c (micromips_opcodes): Use NODS rather than TRAP 
> 	for "alnv.ps".

OK, thanks.

Richard

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-11-15 17:30 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-10-24 21:15 [PATCH] MIPS/opcodes: Fix ALNV.PS's no-delay-slot annotation Maciej W. Rozycki
2011-11-15 17:30 ` Richard Sandiford

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).