qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v2 19/22] target/openrisc: Add print_insn_or1k


From: Richard Henderson
Subject: Re: [Qemu-devel] [PATCH v2 19/22] target/openrisc: Add print_insn_or1k
Date: Wed, 27 Jun 2018 09:15:09 -0700
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.8.0

On 06/27/2018 09:03 AM, Philippe Mathieu-Daudé wrote:
>> +INSN(lwa,    "r%d, %d(r%d)", a->d, a->i, a->a)
>> +INSN(lwz,    "r%d, %d(r%d)", a->d, a->i, a->a)
>> +INSN(lws,    "r%d, %d(r%d)", a->d, a->i, a->a)
>> +INSN(lbz,    "r%d, %d(r%d)", a->d, a->i, a->a)
>> +INSN(lbs,    "r%d, %d(r%d)", a->d, a->i, a->a)
>> +INSN(lhz,    "r%d, %d(r%d)", a->d, a->i, a->a)
>> +INSN(lhs,    "r%d, %d(r%d)", a->d, a->i, a->a)
>> +INSN(swa,    "%d(r%d), r%d", a->i, a->a, a->b)
>> +INSN(sw,     "%d(r%d), r%d", a->i, a->a, a->b)
>> +INSN(sb,     "%d(r%d), r%d", a->i, a->a, a->b)
>> +INSN(sh,     "%d(r%d), r%d", a->i, a->a, a->b)
> 
> You updated to use the offset(reg) notation, ok.

Yes, failure to match the usual format was confusing.

>> +INSN(nop,    "")
> 
> The nop previously displayed 'k', I suppose a left-over from sys/trap.

Yes, real nop has no argument.


r~



reply via email to

[Prev in Thread] Current Thread [Next in Thread]