[PATCH RFC 08/14] m68k/: fix some comment spelling errors

zhaolichang posted 14 patches 5 years, 4 months ago
Maintainers: Sunil Muthuswamy <sunilmut@microsoft.com>, Palmer Dabbelt <palmer@dabbelt.com>, Wenchao Wang <wenchao.wang@intel.com>, Laurent Vivier <laurent@vivier.eu>, Halil Pasic <pasic@linux.ibm.com>, Marcelo Tosatti <mtosatti@redhat.com>, Guan Xuetao <gxt@mprc.pku.edu.cn>, Sagar Karandikar <sagark@eecs.berkeley.edu>, Sarah Harris <S.E.Harris@kent.ac.uk>, Colin Xu <colin.xu@intel.com>, Thomas Huth <thuth@redhat.com>, Aleksandar Markovic <aleksandar.qemu.devel@gmail.com>, David Gibson <david@gibson.dropbear.id.au>, Aurelien Jarno <aurelien@aurel32.net>, Aleksandar Rikalo <aleksandar.rikalo@syrmia.com>, Peter Maydell <peter.maydell@linaro.org>, Cornelia Huck <cohuck@redhat.com>, Bastian Koppelmann <kbastian@mail.uni-paderborn.de>, Jiaxun Yang <jiaxun.yang@flygoat.com>, Alistair Francis <Alistair.Francis@wdc.com>, Christian Borntraeger <borntraeger@de.ibm.com>, Michael Rolnik <mrolnik@gmail.com>, Paolo Bonzini <pbonzini@redhat.com>, Richard Henderson <rth@twiddle.net>, Artyom Tarasenko <atar4qemu@gmail.com>, Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>, Stafford Horne <shorne@gmail.com>, Yoshinori Sato <ysato@users.sourceforge.jp>, David Hildenbrand <david@redhat.com>, "Edgar E. Iglesias" <edgar.iglesias@gmail.com>, Eduardo Habkost <ehabkost@redhat.com>
There is a newer version of this series
[PATCH RFC 08/14] m68k/: fix some comment spelling errors
Posted by zhaolichang 5 years, 4 months ago
I found that there are many spelling errors in the comments of qemu/target/m68k.
I used spellcheck to check the spelling errors and found some errors in the folder.

Signed-off-by: zhaolichang <zhaolichang@huawei.com>
---
 target/m68k/translate.c | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/target/m68k/translate.c b/target/m68k/translate.c
index 3fc67aa..133a404 100644
--- a/target/m68k/translate.c
+++ b/target/m68k/translate.c
@@ -438,7 +438,7 @@ static TCGv gen_addr_index(DisasContext *s, uint16_t ext, TCGv tmp)
 }
 
 /*
- * Handle a base + index + displacement effective addresss.
+ * Handle a base + index + displacement effective address.
  * A NULL_QREG base means pc-relative.
  */
 static TCGv gen_lea_indexed(CPUM68KState *env, DisasContext *s, TCGv base)
@@ -1696,7 +1696,7 @@ static void bcd_add(TCGv dest, TCGv src)
 
     /*
      * t1 = (src + 0x066) + dest + X
-     *    = result with some possible exceding 0x6
+     *    = result with some possible exceeding 0x6
      */
 
     t0 = tcg_const_i32(0x066);
@@ -1706,7 +1706,7 @@ static void bcd_add(TCGv dest, TCGv src)
     tcg_gen_add_i32(t1, t0, dest);
     tcg_gen_add_i32(t1, t1, QREG_CC_X);
 
-    /* we will remove exceding 0x6 where there is no carry */
+    /* we will remove exceeding 0x6 where there is no carry */
 
     /*
      * t0 = (src + 0x0066) ^ dest
@@ -1736,7 +1736,7 @@ static void bcd_add(TCGv dest, TCGv src)
     tcg_temp_free(t0);
 
     /*
-     * remove the exceding 0x6
+     * remove the exceeding 0x6
      * for digits that have not generated a carry
      */
 
@@ -2638,7 +2638,7 @@ DISAS_INSN(negx)
     gen_flush_flags(s); /* compute old Z */
 
     /*
-     * Perform substract with borrow.
+     * Perform subtract with borrow.
      * (X, N) =  -(src + X);
      */
 
@@ -2653,7 +2653,7 @@ DISAS_INSN(negx)
     /*
      * Compute signed-overflow for negation.  The normal formula for
      * subtraction is (res ^ src) & (src ^ dest), but with dest==0
-     * this simplies to res & src.
+     * this simplifies to res & src.
      */
 
     tcg_gen_and_i32(QREG_CC_V, QREG_CC_N, src);
@@ -3159,7 +3159,7 @@ static inline void gen_subx(DisasContext *s, TCGv src, TCGv dest, int opsize)
     gen_flush_flags(s); /* compute old Z */
 
     /*
-     * Perform substract with borrow.
+     * Perform subtract with borrow.
      * (X, N) = dest - (src + X);
      */
 
@@ -3169,7 +3169,7 @@ static inline void gen_subx(DisasContext *s, TCGv src, TCGv dest, int opsize)
     gen_ext(QREG_CC_N, QREG_CC_N, opsize, 1);
     tcg_gen_andi_i32(QREG_CC_X, QREG_CC_X, 1);
 
-    /* Compute signed-overflow for substract.  */
+    /* Compute signed-overflow for subtract.  */
 
     tcg_gen_xor_i32(QREG_CC_V, QREG_CC_N, dest);
     tcg_gen_xor_i32(tmp, dest, src);
-- 
2.26.2.windows.1


Re: [PATCH RFC 08/14] m68k/: fix some comment spelling errors
Posted by David Edmondson 5 years, 4 months ago
On Wednesday, 2020-09-30 at 17:53:15 +08, zhaolichang wrote:

> I found that there are many spelling errors in the comments of qemu/target/m68k.
> I used spellcheck to check the spelling errors and found some errors in the folder.
>
> Signed-off-by: zhaolichang <zhaolichang@huawei.com>

Reviewed-by: David Edmondson <david.edmondson@oracle.com>

> ---
>  target/m68k/translate.c | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/target/m68k/translate.c b/target/m68k/translate.c
> index 3fc67aa..133a404 100644
> --- a/target/m68k/translate.c
> +++ b/target/m68k/translate.c
> @@ -438,7 +438,7 @@ static TCGv gen_addr_index(DisasContext *s, uint16_t ext, TCGv tmp)
>  }
>  
>  /*
> - * Handle a base + index + displacement effective addresss.
> + * Handle a base + index + displacement effective address.
>   * A NULL_QREG base means pc-relative.
>   */
>  static TCGv gen_lea_indexed(CPUM68KState *env, DisasContext *s, TCGv base)
> @@ -1696,7 +1696,7 @@ static void bcd_add(TCGv dest, TCGv src)
>  
>      /*
>       * t1 = (src + 0x066) + dest + X
> -     *    = result with some possible exceding 0x6
> +     *    = result with some possible exceeding 0x6
>       */
>  
>      t0 = tcg_const_i32(0x066);
> @@ -1706,7 +1706,7 @@ static void bcd_add(TCGv dest, TCGv src)
>      tcg_gen_add_i32(t1, t0, dest);
>      tcg_gen_add_i32(t1, t1, QREG_CC_X);
>  
> -    /* we will remove exceding 0x6 where there is no carry */
> +    /* we will remove exceeding 0x6 where there is no carry */
>  
>      /*
>       * t0 = (src + 0x0066) ^ dest
> @@ -1736,7 +1736,7 @@ static void bcd_add(TCGv dest, TCGv src)
>      tcg_temp_free(t0);
>  
>      /*
> -     * remove the exceding 0x6
> +     * remove the exceeding 0x6
>       * for digits that have not generated a carry
>       */
>  
> @@ -2638,7 +2638,7 @@ DISAS_INSN(negx)
>      gen_flush_flags(s); /* compute old Z */
>  
>      /*
> -     * Perform substract with borrow.
> +     * Perform subtract with borrow.
>       * (X, N) =  -(src + X);
>       */
>  
> @@ -2653,7 +2653,7 @@ DISAS_INSN(negx)
>      /*
>       * Compute signed-overflow for negation.  The normal formula for
>       * subtraction is (res ^ src) & (src ^ dest), but with dest==0
> -     * this simplies to res & src.
> +     * this simplifies to res & src.
>       */
>  
>      tcg_gen_and_i32(QREG_CC_V, QREG_CC_N, src);
> @@ -3159,7 +3159,7 @@ static inline void gen_subx(DisasContext *s, TCGv src, TCGv dest, int opsize)
>      gen_flush_flags(s); /* compute old Z */
>  
>      /*
> -     * Perform substract with borrow.
> +     * Perform subtract with borrow.
>       * (X, N) = dest - (src + X);
>       */
>  
> @@ -3169,7 +3169,7 @@ static inline void gen_subx(DisasContext *s, TCGv src, TCGv dest, int opsize)
>      gen_ext(QREG_CC_N, QREG_CC_N, opsize, 1);
>      tcg_gen_andi_i32(QREG_CC_X, QREG_CC_X, 1);
>  
> -    /* Compute signed-overflow for substract.  */
> +    /* Compute signed-overflow for subtract.  */
>  
>      tcg_gen_xor_i32(QREG_CC_V, QREG_CC_N, dest);
>      tcg_gen_xor_i32(tmp, dest, src);
> -- 
> 2.26.2.windows.1

dme.
-- 
Don't you know you're never going to get to France.

Re: [PATCH RFC 08/14] m68k/: fix some comment spelling errors
Posted by Laurent Vivier 5 years, 4 months ago
Le 30/09/2020 à 11:53, zhaolichang a écrit :
> I found that there are many spelling errors in the comments of qemu/target/m68k.
> I used spellcheck to check the spelling errors and found some errors in the folder.
> 
> Signed-off-by: zhaolichang <zhaolichang@huawei.com>
> ---
>  target/m68k/translate.c | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/target/m68k/translate.c b/target/m68k/translate.c
> index 3fc67aa..133a404 100644
> --- a/target/m68k/translate.c
> +++ b/target/m68k/translate.c
> @@ -438,7 +438,7 @@ static TCGv gen_addr_index(DisasContext *s, uint16_t ext, TCGv tmp)
>  }
>  
>  /*
> - * Handle a base + index + displacement effective addresss.
> + * Handle a base + index + displacement effective address.
>   * A NULL_QREG base means pc-relative.
>   */
>  static TCGv gen_lea_indexed(CPUM68KState *env, DisasContext *s, TCGv base)
> @@ -1696,7 +1696,7 @@ static void bcd_add(TCGv dest, TCGv src)
>  
>      /*
>       * t1 = (src + 0x066) + dest + X
> -     *    = result with some possible exceding 0x6
> +     *    = result with some possible exceeding 0x6
>       */
>  
>      t0 = tcg_const_i32(0x066);
> @@ -1706,7 +1706,7 @@ static void bcd_add(TCGv dest, TCGv src)
>      tcg_gen_add_i32(t1, t0, dest);
>      tcg_gen_add_i32(t1, t1, QREG_CC_X);
>  
> -    /* we will remove exceding 0x6 where there is no carry */
> +    /* we will remove exceeding 0x6 where there is no carry */
>  
>      /*
>       * t0 = (src + 0x0066) ^ dest
> @@ -1736,7 +1736,7 @@ static void bcd_add(TCGv dest, TCGv src)
>      tcg_temp_free(t0);
>  
>      /*
> -     * remove the exceding 0x6
> +     * remove the exceeding 0x6
>       * for digits that have not generated a carry
>       */
>  
> @@ -2638,7 +2638,7 @@ DISAS_INSN(negx)
>      gen_flush_flags(s); /* compute old Z */
>  
>      /*
> -     * Perform substract with borrow.
> +     * Perform subtract with borrow.
>       * (X, N) =  -(src + X);
>       */
>  
> @@ -2653,7 +2653,7 @@ DISAS_INSN(negx)
>      /*
>       * Compute signed-overflow for negation.  The normal formula for
>       * subtraction is (res ^ src) & (src ^ dest), but with dest==0
> -     * this simplies to res & src.
> +     * this simplifies to res & src.
>       */
>  
>      tcg_gen_and_i32(QREG_CC_V, QREG_CC_N, src);
> @@ -3159,7 +3159,7 @@ static inline void gen_subx(DisasContext *s, TCGv src, TCGv dest, int opsize)
>      gen_flush_flags(s); /* compute old Z */
>  
>      /*
> -     * Perform substract with borrow.
> +     * Perform subtract with borrow.
>       * (X, N) = dest - (src + X);
>       */
>  
> @@ -3169,7 +3169,7 @@ static inline void gen_subx(DisasContext *s, TCGv src, TCGv dest, int opsize)
>      gen_ext(QREG_CC_N, QREG_CC_N, opsize, 1);
>      tcg_gen_andi_i32(QREG_CC_X, QREG_CC_X, 1);
>  
> -    /* Compute signed-overflow for substract.  */
> +    /* Compute signed-overflow for subtract.  */
>  
>      tcg_gen_xor_i32(QREG_CC_V, QREG_CC_N, dest);
>      tcg_gen_xor_i32(tmp, dest, src);
> 

Reviewed-by: Laurent Vivier <laurent@vivier>

Re: [PATCH RFC 08/14] m68k/: fix some comment spelling errors
Posted by Philippe Mathieu-Daudé 5 years, 4 months ago
On 9/30/20 12:26 PM, Laurent Vivier wrote:
> Le 30/09/2020 à 11:53, zhaolichang a écrit :
>> I found that there are many spelling errors in the comments of qemu/target/m68k.
>> I used spellcheck to check the spelling errors and found some errors in the folder.
>>
>> Signed-off-by: zhaolichang <zhaolichang@huawei.com>
>> ---
>>  target/m68k/translate.c | 16 ++++++++--------
>>  1 file changed, 8 insertions(+), 8 deletions(-)
>>
>> diff --git a/target/m68k/translate.c b/target/m68k/translate.c
>> index 3fc67aa..133a404 100644
>> --- a/target/m68k/translate.c
>> +++ b/target/m68k/translate.c
>> @@ -438,7 +438,7 @@ static TCGv gen_addr_index(DisasContext *s, uint16_t ext, TCGv tmp)
>>  }
>>  
>>  /*
>> - * Handle a base + index + displacement effective addresss.
>> + * Handle a base + index + displacement effective address.
>>   * A NULL_QREG base means pc-relative.
>>   */
>>  static TCGv gen_lea_indexed(CPUM68KState *env, DisasContext *s, TCGv base)
>> @@ -1696,7 +1696,7 @@ static void bcd_add(TCGv dest, TCGv src)
>>  
>>      /*
>>       * t1 = (src + 0x066) + dest + X
>> -     *    = result with some possible exceding 0x6
>> +     *    = result with some possible exceeding 0x6
>>       */
>>  
>>      t0 = tcg_const_i32(0x066);
>> @@ -1706,7 +1706,7 @@ static void bcd_add(TCGv dest, TCGv src)
>>      tcg_gen_add_i32(t1, t0, dest);
>>      tcg_gen_add_i32(t1, t1, QREG_CC_X);
>>  
>> -    /* we will remove exceding 0x6 where there is no carry */
>> +    /* we will remove exceeding 0x6 where there is no carry */
>>  
>>      /*
>>       * t0 = (src + 0x0066) ^ dest
>> @@ -1736,7 +1736,7 @@ static void bcd_add(TCGv dest, TCGv src)
>>      tcg_temp_free(t0);
>>  
>>      /*
>> -     * remove the exceding 0x6
>> +     * remove the exceeding 0x6
>>       * for digits that have not generated a carry
>>       */
>>  
>> @@ -2638,7 +2638,7 @@ DISAS_INSN(negx)
>>      gen_flush_flags(s); /* compute old Z */
>>  
>>      /*
>> -     * Perform substract with borrow.
>> +     * Perform subtract with borrow.
>>       * (X, N) =  -(src + X);
>>       */
>>  
>> @@ -2653,7 +2653,7 @@ DISAS_INSN(negx)
>>      /*
>>       * Compute signed-overflow for negation.  The normal formula for
>>       * subtraction is (res ^ src) & (src ^ dest), but with dest==0
>> -     * this simplies to res & src.
>> +     * this simplifies to res & src.
>>       */
>>  
>>      tcg_gen_and_i32(QREG_CC_V, QREG_CC_N, src);
>> @@ -3159,7 +3159,7 @@ static inline void gen_subx(DisasContext *s, TCGv src, TCGv dest, int opsize)
>>      gen_flush_flags(s); /* compute old Z */
>>  
>>      /*
>> -     * Perform substract with borrow.
>> +     * Perform subtract with borrow.
>>       * (X, N) = dest - (src + X);
>>       */
>>  
>> @@ -3169,7 +3169,7 @@ static inline void gen_subx(DisasContext *s, TCGv src, TCGv dest, int opsize)
>>      gen_ext(QREG_CC_N, QREG_CC_N, opsize, 1);
>>      tcg_gen_andi_i32(QREG_CC_X, QREG_CC_X, 1);
>>  
>> -    /* Compute signed-overflow for substract.  */
>> +    /* Compute signed-overflow for subtract.  */
>>  
>>      tcg_gen_xor_i32(QREG_CC_V, QREG_CC_N, dest);
>>      tcg_gen_xor_i32(tmp, dest, src);
>>
> 
> Reviewed-by: Laurent Vivier <laurent@vivier>

Reviewed-by: Laurent Vivier <laurent@vivier.eu>

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

Re: [PATCH RFC 08/14] m68k/: fix some comment spelling errors
Posted by Laurent Vivier 5 years, 4 months ago
Le 30/09/2020 à 18:03, Philippe Mathieu-Daudé a écrit :
> On 9/30/20 12:26 PM, Laurent Vivier wrote:
>> Le 30/09/2020 à 11:53, zhaolichang a écrit :
>>> I found that there are many spelling errors in the comments of qemu/target/m68k.
>>> I used spellcheck to check the spelling errors and found some errors in the folder.
>>>
>>> Signed-off-by: zhaolichang <zhaolichang@huawei.com>
>>> ---
>>>  target/m68k/translate.c | 16 ++++++++--------
>>>  1 file changed, 8 insertions(+), 8 deletions(-)
>>>
>>> diff --git a/target/m68k/translate.c b/target/m68k/translate.c
>>> index 3fc67aa..133a404 100644
>>> --- a/target/m68k/translate.c
>>> +++ b/target/m68k/translate.c
>>> @@ -438,7 +438,7 @@ static TCGv gen_addr_index(DisasContext *s, uint16_t ext, TCGv tmp)
>>>  }
>>>  
>>>  /*
>>> - * Handle a base + index + displacement effective addresss.
>>> + * Handle a base + index + displacement effective address.
>>>   * A NULL_QREG base means pc-relative.
>>>   */
>>>  static TCGv gen_lea_indexed(CPUM68KState *env, DisasContext *s, TCGv base)
>>> @@ -1696,7 +1696,7 @@ static void bcd_add(TCGv dest, TCGv src)
>>>  
>>>      /*
>>>       * t1 = (src + 0x066) + dest + X
>>> -     *    = result with some possible exceding 0x6
>>> +     *    = result with some possible exceeding 0x6
>>>       */
>>>  
>>>      t0 = tcg_const_i32(0x066);
>>> @@ -1706,7 +1706,7 @@ static void bcd_add(TCGv dest, TCGv src)
>>>      tcg_gen_add_i32(t1, t0, dest);
>>>      tcg_gen_add_i32(t1, t1, QREG_CC_X);
>>>  
>>> -    /* we will remove exceding 0x6 where there is no carry */
>>> +    /* we will remove exceeding 0x6 where there is no carry */
>>>  
>>>      /*
>>>       * t0 = (src + 0x0066) ^ dest
>>> @@ -1736,7 +1736,7 @@ static void bcd_add(TCGv dest, TCGv src)
>>>      tcg_temp_free(t0);
>>>  
>>>      /*
>>> -     * remove the exceding 0x6
>>> +     * remove the exceeding 0x6
>>>       * for digits that have not generated a carry
>>>       */
>>>  
>>> @@ -2638,7 +2638,7 @@ DISAS_INSN(negx)
>>>      gen_flush_flags(s); /* compute old Z */
>>>  
>>>      /*
>>> -     * Perform substract with borrow.
>>> +     * Perform subtract with borrow.
>>>       * (X, N) =  -(src + X);
>>>       */
>>>  
>>> @@ -2653,7 +2653,7 @@ DISAS_INSN(negx)
>>>      /*
>>>       * Compute signed-overflow for negation.  The normal formula for
>>>       * subtraction is (res ^ src) & (src ^ dest), but with dest==0
>>> -     * this simplies to res & src.
>>> +     * this simplifies to res & src.
>>>       */
>>>  
>>>      tcg_gen_and_i32(QREG_CC_V, QREG_CC_N, src);
>>> @@ -3159,7 +3159,7 @@ static inline void gen_subx(DisasContext *s, TCGv src, TCGv dest, int opsize)
>>>      gen_flush_flags(s); /* compute old Z */
>>>  
>>>      /*
>>> -     * Perform substract with borrow.
>>> +     * Perform subtract with borrow.
>>>       * (X, N) = dest - (src + X);
>>>       */
>>>  
>>> @@ -3169,7 +3169,7 @@ static inline void gen_subx(DisasContext *s, TCGv src, TCGv dest, int opsize)
>>>      gen_ext(QREG_CC_N, QREG_CC_N, opsize, 1);
>>>      tcg_gen_andi_i32(QREG_CC_X, QREG_CC_X, 1);
>>>  
>>> -    /* Compute signed-overflow for substract.  */
>>> +    /* Compute signed-overflow for subtract.  */
>>>  
>>>      tcg_gen_xor_i32(QREG_CC_V, QREG_CC_N, dest);
>>>      tcg_gen_xor_i32(tmp, dest, src);
>>>
>>
>> Reviewed-by: Laurent Vivier <laurent@vivier>
> 
> Reviewed-by: Laurent Vivier <laurent@vivier.eu>

Yes, thanks.

> 
> Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> 

Laurent