[PATCHv2 1/3] opcodes/arc: Move instruction length logic to new function

Claudiu Zissulescu Claudiu.Zissulescu@synopsys.com
Tue Apr 12 12:27:00 GMT 2016


Hi Andrew,

> +    case bfd_mach_arc_nps400:
> +    case bfd_mach_arc_arc700:
> +    case bfd_mach_arc_arc600:
> +      len = (major_opcode > 0xb) ? 2 : 4;
> +      break;

Ok, this solves the hole between 0x07 and 0x0b existing in your former proposal. I think this implementation is safer than the older proposal and nicer than original code.

Cheers,
Claudiu



More information about the Binutils mailing list