Re: [PATCH v2 62/76] target/microblaze: Tidy mb_cpu_dump_state

[Philippe Mathieu-Daudé]

Le mar. 1 sept. 2020 00:16, Richard Henderson <richard.henderson@linaro.org> a écrit :

On 8/31/20 2:08 PM, Philippe Mathieu-Daudé wrote:
> Hi Richard, 
>
> Le ven. 28 août 2020 16:46, Richard Henderson <richard.henderson@linaro.org
> <mailto:richard.henderson@linaro.org>> a écrit :
>
>     Using lookup_symbol is quite slow; remove that.  Decode the
>
>
> Can we add some runtime option to enable its use instead?

We do the lookup with -d in_asm.
Do we really need it with -d cpu as well?

The -d cpu can be called many millions of times more than -d in_asm, and the
two are relatively easy to correlate.

Ah I missed that. This is fine then!

Thanks, 

Phil.

r~