Re: [Qemu-devel] [PATCH v2 14/17] target/openrisc: convert to TranslatorOps

2018-04-09 Thread Richard Henderson
On 04/07/2018 04:19 AM, Emilio G. Cota wrote: > Notes: > > - Changed the num_insns test in insn_start to check for > dc->base.num_insns > 1, since when tb_start is first > called in a TB, base.num_insns is already set to 1. > > - Removed DISAS_NEXT from the switch in tb_stop; use > DISAS_TO

[Qemu-devel] [PATCH v2 14/17] target/openrisc: convert to TranslatorOps

2018-04-06 Thread Emilio G. Cota
Notes: - Changed the num_insns test in insn_start to check for dc->base.num_insns > 1, since when tb_start is first called in a TB, base.num_insns is already set to 1. - Removed DISAS_NEXT from the switch in tb_stop; use DISAS_TOO_MANY instead. - Added an assert_not_reached on tb_stop for