Hi,
> +static int
> +arc_insn_length (unsigned isa_mask, bfd_byte msb,
> + bfd_byte lsb ATTRIBUTE_UNUSED)
> +{
> + static int tbl [32]; /* 0x00 -> 0x1f */
> + static int tbl_init = 0;
Please use bfd_boolean instead of int for tbl_init.
Otherwise, it seems sane to me.
Best,
Claudiu