On 5/23/22 05:53, Peter Maydell wrote:
On Sat, 21 May 2022 at 01:04, Richard Henderson
<richard.hender...@linaro.org> wrote:

We've already loaded cs->env_ptr into a local variable; use it.
Since env is unconditionally used, we don't need a dummy use.

Signed-off-by: Richard Henderson <richard.hender...@linaro.org>
---
  target/arm/common-semi-target.h   |  62 ++++++++++++++++++
  target/riscv/common-semi-target.h |  50 +++++++++++++++
  semihosting/arm-compat-semi.c     | 101 ++----------------------------
  3 files changed, 116 insertions(+), 97 deletions(-)
  create mode 100644 target/arm/common-semi-target.h
  create mode 100644 target/riscv/common-semi-target.h

Commit message doesn't match contents -- two changes accidentally
squashed together here ?

Yep, that's exactly it.


r~

Reply via email to