This is the mail archive of the binutils@sources.redhat.com mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] ia64 segfault on nop in X-slot of MLX bundle


On Mon, 2005-01-24 at 02:35, Jan Beulich wrote:
> 	* config/tc-ia64.c (emit_one_bundle): Allow .x suffix when
> resolving
> 	dynamic insns.

OK.

nop.x wasn't in the original architecture.  My recollection is that brl
was added first, and then nop.x was added later, and when that happened,
we missed the fact that this string needed to change to also allow x
unit instructions.

I'd write the ChangeLog entry more like
	Change "?imbf??" to "?ibmfxx".
which describes what changed, instead of why it was changed.
-- 
Jim Wilson, GNU Tools Support, http://www.SpecifixInc.com



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]