qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCHv2 1/2] translator: merge max_insns into DisasCon


From: Richard Henderson
Subject: Re: [Qemu-devel] [PATCHv2 1/2] translator: merge max_insns into DisasContextBase
Date: Wed, 21 Feb 2018 10:19:05 -0800
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0

On 02/19/2018 06:26 PM, Emilio G. Cota wrote:
> @@ -68,6 +69,7 @@ typedef struct DisasContextBase {
>      target_ulong pc_next;
>      DisasJumpType is_jmp;
>      unsigned int num_insns;
> +    int max_insns;
>      bool singlestep_enabled;
>  } DisasContextBase;

We really should pick the same type for max_insns and num_insns, which ever
type we settle on.  I can't see how we can go wrong with unsigned...


r~



reply via email to

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