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

James E Wilson wilson@specifixinc.com
Thu Jan 27 03:39:00 GMT 2005


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




More information about the Binutils mailing list