|
From: | Richard Henderson |
Subject: | Re: [PATCH 17/21] Hexagon (target/hexagon) Move new_value to DisasContext |
Date: | Thu, 27 Apr 2023 12:03:23 +0100 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.10.0 |
On 4/27/23 12:01, Richard Henderson wrote:
On 4/26/23 01:42, Taylor Simpson wrote:+ for (i = 0; i < TOTAL_PER_THREAD_REGS; i++) { + ctx->new_value[i] = NULL; + }Perhaps memset(ctx->new_value, 0, sizeof(ctx->new_value)); Though probably the compiler would make that transformation. Or perhaps - DisasContext ctx; + DisasContext ctx = { 0 }; in gen_intermediate_code, and eliminate other 0 init in gen_start_packet?
Duh, start packet is called more than once per TB. r~
[Prev in Thread] | Current Thread | [Next in Thread] |