[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 04/11] disas: Move host asm annotations to tb_gen_code
From: |
Alex Bennée |
Subject: |
Re: [PATCH v4 04/11] disas: Move host asm annotations to tb_gen_code |
Date: |
Tue, 22 Sep 2020 10:50:34 +0100 |
User-agent: |
mu4e 1.5.5; emacs 28.0.50 |
Philippe Mathieu-Daudé <f4bug@amsat.org> writes:
> On 9/21/20 9:53 PM, Richard Henderson wrote:
>> On 9/21/20 12:29 PM, Alex Bennée wrote:
>>>
>>> Richard Henderson <richard.henderson@linaro.org> writes:
>>>
>>>> Instead of creating GStrings and passing them into log_disas,
>>>> just print the annotations directly in tb_gen_code.
>>>>
>>>> Fix the annotations for the slow paths of the TB, after the
>>>> part implementing the final guest instruction.
>>>>
>>>> Reviewed-by: Thomas Huth <thuth@redhat.com>
>>>> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
>>>
>>> I guess what we loose in the inline annotation we gain in simpler code.
>>> We can always grep stuff out of the logs if we need to:
>>
>> What information do you think we're losing?
>
> This in tb_gen_code()?
>
> note = g_string_new("[tb header & initial instruction]");
>
> g_string_printf(note, "[guest addr: " TARGET_FMT_lx "]",
> tcg_ctx->gen_insn_data[insn][0]);
We are not loosing information - just it's placement is slightly
different. It's nothing you can't work around.
--
Alex Bennée
- [PATCH v4 08/11] disas: Configure capstone for aarch64 host without libvixl, (continued)
[PATCH v4 09/11] disas: Split out capstone code to disas/capstone.c, Richard Henderson, 2020/09/21
[PATCH v4 10/11] disas: Enable capstone disassembly for s390x, Richard Henderson, 2020/09/21
[PATCH v4 11/11] disas/capstone: Add skipdata hook for s390x, Richard Henderson, 2020/09/21
Re: [PATCH v4 00/11] capstone + disassembler patches, no-reply, 2020/09/21