Re: [PATCH v3 18/19] target/microblaze: Put MicroBlazeCPUConfig into DisasContext

[Philippe Mathieu-Daudé]

On 9/4/20 9:08 PM, Richard Henderson wrote:
> The bulk of the translator should not have access to the
> complete cpu state, to avoid the temptation to examine bits
> that are in run time, but not translation time context.
> 
> We do need access to the constant cpu configuration, and
> that is sufficient, so put that into DisasContext.
> 
> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
> ---
>  target/microblaze/translate.c | 24 ++++++++++++------------
>  1 file changed, 12 insertions(+), 12 deletions(-)

Nice.

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