On 11/4/23 03:04, Richard Henderson wrote:
While the old type was correct in the ideal sense,
some ABIs require the argument to be zero-extended.
Using uint32_t for all such values is a decent compromise.

Signed-off-by: Richard Henderson <richard.hender...@linaro.org>
---
  include/tcg/tcg-ldst.h | 10 +++++++---
  accel/tcg/cputlb.c     |  6 +++---
  2 files changed, 10 insertions(+), 6 deletions(-)


-void helper_ret_stb_mmu(CPUArchState *env, target_ulong addr, uint8_t val,
+/*
+ * Value extended to at least uint32_t, so that some abis do not require

s/abis/ABIs/

Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org>

+ * zero-extension from uint8_t or uint16_t.
+ */

Reply via email to