On 8/22/21 5:32 AM, Peter Maydell wrote:
On Sun, 22 Aug 2021 at 08:59, Richard Henderson
<[email protected]> wrote:
On 8/20/21 3:14 AM, Peter Maydell wrote:
@@ -377,11 +381,11 @@ static uint64_t tci_qemu_ld(CPUArchState *env,
target_ulong taddr,
static void tci_qemu_st(CPUArchState *env, target_ulong taddr, uint64_t val,
MemOpIdx oi, const void *tb_ptr)
{
- MemOp mop = get_memop(oi) & (MO_BSWAP | MO_SSIZE);
+ MemOp mop = get_memop(oi);
uintptr_t ra = (uintptr_t)tb_ptr;
Don't you need this bit in tci_qemu_st() as well ?
Which bit isn't present in st as well?
There's missing hunks in your reply, but afaics they're the same.
https://patchew.org/QEMU/[email protected]/[email protected]/
I had the function name wrong, but only the tci_qemu_st() change
has this bit:
- MemOp mop = get_memop(oi) & (MO_BSWAP | MO_SSIZE);
+ MemOp mop = get_memop(oi);
Ah yes, thanks.
r~
