On 16 December 2013 10:11, C Fontana <claudio.font...@linaro.org> wrote: > Sorry for noticing only now, > but there is a missing return here as well I think: > >> static void disas_add_sub_imm(DisasContext *s, uint32_t insn)
>> + default: >> + unallocated_encoding(s); > > should return; here I think. Yes, fixed. -- PMM