On 12/1/26 16:40, Anton Johansson wrote:
MemLog::width is a uint8_t value mapped to a TCGv (32 bit), the only
reason this currently works is because MemLog::width is padded to 32
bits. Widen the field to uint32_t and fix the size of the TCGv
operations as well. Use uint32_t when referencing and passing around
the field, as valid values are asserted in commit_store().
Signed-off-by: Anton Johansson <[email protected]>
--
Changes in v2:
- Removed truncation to uint8_t, valid values of 1,2,4,8 are checked in
commit_store() already.
---
target/hexagon/cpu.h | 2 +-
target/hexagon/genptr.h | 2 +-
target/hexagon/op_helper.h | 4 ++--
target/hexagon/translate.h | 2 +-
target/hexagon/genptr.c | 6 +++---
target/hexagon/op_helper.c | 8 ++++----
target/hexagon/translate.c | 4 ++--
7 files changed, 14 insertions(+), 14 deletions(-)
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
