RE: [PATCH 1/9] target/i386: silence the compiler warnings in gen_shiftd_rm_T1

2020-10-28 Thread Chenqun (kuhn)
> Robot ; Paolo Bonzini ; > Richard Henderson > Subject: Re: [PATCH 1/9] target/i386: silence the compiler warnings in > gen_shiftd_rm_T1 > > On 28/10/2020 16.31, Richard Henderson wrote: > > On 10/28/20 5:57 AM, Thomas Huth wrote: > >> On 28/10/2020 05.18, Chen Qun

Re: [PATCH 1/9] target/i386: silence the compiler warnings in gen_shiftd_rm_T1

2020-10-28 Thread Thomas Huth
On 28/10/2020 16.31, Richard Henderson wrote: > On 10/28/20 5:57 AM, Thomas Huth wrote: >> On 28/10/2020 05.18, Chen Qun wrote: >>> The current "#ifdef TARGET_X86_64" statement affects >>> the compiler's determination of fall through. >>> >>> When using -Wimplicit-fallthrough in our CFLAGS, the com

Re: [PATCH 1/9] target/i386: silence the compiler warnings in gen_shiftd_rm_T1

2020-10-28 Thread Richard Henderson
On 10/27/20 9:18 PM, Chen Qun wrote: > The current "#ifdef TARGET_X86_64" statement affects > the compiler's determination of fall through. > > When using -Wimplicit-fallthrough in our CFLAGS, the compiler showed warning: > target/i386/translate.c: In function ‘gen_shiftd_rm_T1’: > target/i386/tra

Re: [PATCH 1/9] target/i386: silence the compiler warnings in gen_shiftd_rm_T1

2020-10-28 Thread Richard Henderson
On 10/28/20 5:57 AM, Thomas Huth wrote: > On 28/10/2020 05.18, Chen Qun wrote: >> The current "#ifdef TARGET_X86_64" statement affects >> the compiler's determination of fall through. >> >> When using -Wimplicit-fallthrough in our CFLAGS, the compiler showed warning: >> target/i386/translate.c: In

Re: [PATCH 1/9] target/i386: silence the compiler warnings in gen_shiftd_rm_T1

2020-10-28 Thread Philippe Mathieu-Daudé
+Tony On 10/28/20 1:57 PM, Thomas Huth wrote: > On 28/10/2020 05.18, Chen Qun wrote: >> The current "#ifdef TARGET_X86_64" statement affects >> the compiler's determination of fall through. >> >> When using -Wimplicit-fallthrough in our CFLAGS, the compiler showed warning: >> target/i386/translate

Re: [PATCH 1/9] target/i386: silence the compiler warnings in gen_shiftd_rm_T1

2020-10-28 Thread Thomas Huth
On 28/10/2020 05.18, Chen Qun wrote: > The current "#ifdef TARGET_X86_64" statement affects > the compiler's determination of fall through. > > When using -Wimplicit-fallthrough in our CFLAGS, the compiler showed warning: > target/i386/translate.c: In function ‘gen_shiftd_rm_T1’: > target/i386/tra

[PATCH 1/9] target/i386: silence the compiler warnings in gen_shiftd_rm_T1

2020-10-27 Thread Chen Qun
The current "#ifdef TARGET_X86_64" statement affects the compiler's determination of fall through. When using -Wimplicit-fallthrough in our CFLAGS, the compiler showed warning: target/i386/translate.c: In function ‘gen_shiftd_rm_T1’: target/i386/translate.c:1773:12: warning: this statement may fal