On 6/19/20 10:03 AM, Peter Maydell wrote: > Since commit ba3e7926691ed3 it has been unnecessary for target code > to call gen_io_end() after an IO instruction in icount mode; it is > sufficient to call gen_io_start() before it and to force the end of > the TB. > > Many now-unnecessary calls to gen_io_end() were removed in commit > 9e9b10c6491153b, but some were missed or accidentally added later. > Remove unneeded calls from the arm target: > > * the call in the handling of exception-return-via-LDM is > unnecessary, and the code is already forcing end-of-TB > * the call in the VFP access check code is more complicated: > we weren't ending the TB, so we need to add the code to > force that by setting DISAS_UPDATE > * the doc comment for ARM_CP_IO doesn't need to mention > gen_io_end() any more > > Signed-off-by: Peter Maydell <peter.mayd...@linaro.org> > --- > target/arm/cpu.h | 2 +- > target/arm/translate-vfp.inc.c | 7 +++---- > target/arm/translate.c | 3 --- > 3 files changed, 4 insertions(+), 8 deletions(-)
Reviewed-by: Richard Henderson <richard.hender...@linaro.org> r~
