qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH v4 06/33] tcg-aarch64: Merge enum aarch64_srr_op


From: Richard Henderson
Subject: Re: [Qemu-devel] [PATCH v4 06/33] tcg-aarch64: Merge enum aarch64_srr_opc with AArch64Insn
Date: Mon, 16 Sep 2013 08:06:13 -0700
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130805 Thunderbird/17.0.8

On 09/16/2013 12:56 AM, Claudio Fontana wrote:
>> >      case INDEX_op_shl_i64:
>> >      case INDEX_op_shl_i32:
>> > -        if (c2) {    /* LSL / UBFM Wd, Wn, (32 - m) */
>> > +        if (c2) {
>> >              tcg_out_shl(s, ext, a0, a1, a2);
>> > -        } else {                /* LSL / LSLV */
>> > -            tcg_out_shiftrot_reg(s, SRR_SHL, ext, a0, a1, a2);
>> > +        } else {
>> > +            tcg_out_shiftrot_reg(s, INSN_LSLV, ext, a0, a1, a2);
>> >          }
>> >          break;
> Any reason you strip all comments out?
> They are supposed to hint the reader about how the tcg operation is 
> implemented.
> 

Well, LSLV is now there in the code in the form of INSN_LSLV.  The comment does
nothing additional to hint at what's going on; indeed, "LSL / LSLV" is more
confusing than not.

As for tcg_out_shl, comments about its implementation are with that function.


r~



reply via email to

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