diff mbox series

target/m68k: Fix gen_load_fp for OS_LONG

Message ID 20230508140857.137405-1-richard.henderson@linaro.org
State Superseded
Headers show
Series target/m68k: Fix gen_load_fp for OS_LONG | expand

Commit Message

Richard Henderson May 8, 2023, 2:08 p.m. UTC
Case was accidentally dropped in b7a94da9550b.

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
 target/m68k/translate.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Laurent Vivier May 8, 2023, 4:06 p.m. UTC | #1
Le 08/05/2023 à 16:08, Richard Henderson a écrit :
> Case was accidentally dropped in b7a94da9550b.
> 
> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
> ---
>   target/m68k/translate.c | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/target/m68k/translate.c b/target/m68k/translate.c
> index 744eb3748b..44d852b106 100644
> --- a/target/m68k/translate.c
> +++ b/target/m68k/translate.c
> @@ -959,6 +959,7 @@ static void gen_load_fp(DisasContext *s, int opsize, TCGv addr, TCGv_ptr fp,
>       switch (opsize) {
>       case OS_BYTE:
>       case OS_WORD:
> +    case OS_LONG:
>           tcg_gen_qemu_ld_tl(tmp, addr, index, opsize | MO_SIGN | MO_TE);
>           gen_helper_exts32(cpu_env, fp, tmp);
>           break;

Tested-by: Laurent Vivier <laurent@vivier.eu>
Reviewed-by: Laurent Vivier <laurent@vivier.eu>
Richard Henderson May 9, 2023, 2:54 p.m. UTC | #2
On 5/8/23 17:06, Laurent Vivier wrote:
> Le 08/05/2023 à 16:08, Richard Henderson a écrit :
>> Case was accidentally dropped in b7a94da9550b.
>>
>> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
>> ---
>>   target/m68k/translate.c | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/target/m68k/translate.c b/target/m68k/translate.c
>> index 744eb3748b..44d852b106 100644
>> --- a/target/m68k/translate.c
>> +++ b/target/m68k/translate.c
>> @@ -959,6 +959,7 @@ static void gen_load_fp(DisasContext *s, int opsize, TCGv addr, 
>> TCGv_ptr fp,
>>       switch (opsize) {
>>       case OS_BYTE:
>>       case OS_WORD:
>> +    case OS_LONG:
>>           tcg_gen_qemu_ld_tl(tmp, addr, index, opsize | MO_SIGN | MO_TE);
>>           gen_helper_exts32(cpu_env, fp, tmp);
>>           break;
> 
> Tested-by: Laurent Vivier <laurent@vivier.eu>
> Reviewed-by: Laurent Vivier <laurent@vivier.eu>

Queued to tcg-next.


r~
Philippe Mathieu-Daudé May 9, 2023, 3:19 p.m. UTC | #3
On 8/5/23 16:08, Richard Henderson wrote:
> Case was accidentally dropped in b7a94da9550b.
> 
> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
> ---
>   target/m68k/translate.c | 1 +
>   1 file changed, 1 insertion(+)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
diff mbox series

Patch

diff --git a/target/m68k/translate.c b/target/m68k/translate.c
index 744eb3748b..44d852b106 100644
--- a/target/m68k/translate.c
+++ b/target/m68k/translate.c
@@ -959,6 +959,7 @@  static void gen_load_fp(DisasContext *s, int opsize, TCGv addr, TCGv_ptr fp,
     switch (opsize) {
     case OS_BYTE:
     case OS_WORD:
+    case OS_LONG:
         tcg_gen_qemu_ld_tl(tmp, addr, index, opsize | MO_SIGN | MO_TE);
         gen_helper_exts32(cpu_env, fp, tmp);
         break;