Re: [PATCH v2 21/50] target/i386: Reduce DisasContext.override to int8_t

[Paolo Bonzini]

On 14/05/21 17:13, Richard Henderson wrote:
> The range of values is -1 (none) to 5 (R_GS).
>
> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
> ---
>   target/i386/tcg/translate.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/target/i386/tcg/translate.c b/target/i386/tcg/translate.c
> index ca7f0a8cf4..44a28858aa 100644
> --- a/target/i386/tcg/translate.c
> +++ b/target/i386/tcg/translate.c
> @@ -77,7 +77,7 @@ typedef struct DisasContext {
>       DisasContextBase base;
>   
>       /* current insn context */
> -    int override; /* -1 if no override */
> +    int8_t override; /* -1 if no override, else R_CS, R_DS, etc */
>       int prefix;
>       MemOp aflag;
>       MemOp dflag;

Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>