On 1/15/20 1:03 AM, Laurent Vivier wrote:
> + if (dc->base.singlestep_enabled) {
> + tcg_gen_movi_i32(QREG_PC, dc->pc);
> + gen_helper_raise_exception(cpu_env, tcg_const_i32(EXCP_DEBUG));
This leaks the temporary, and so not quite ideal. It would be of more concern
if the rest of the m68k port was audited, so that you could turn on leak
detection.
I would suggest routing all calls to gen_helper_raise_exception through a
function (gen_raise_exception, usually), which takes an int argument, and does
all of the TCGv_i32 management internally.
But otherwise,
Reviewed-by: Richard Henderson <richard.hender...@linaro.org>
r~
