On 12/05/2013 11:14 AM, Peter Maydell wrote:
> On 4 December 2013 21:55, Richard Henderson <r...@twiddle.net> wrote:
>> On 12/05/2013 08:33 AM, Peter Maydell wrote:
>>> @@ -680,6 +720,7 @@ void gen_intermediate_code_internal_a64(ARMCPU *cpu,
>>>      dc->condjmp = 0;
>>>
>>>      dc->aarch64 = 1;
>>> +    dc->tmp_a64_count = 0;
>>>      dc->thumb = 0;
>>>      dc->bswap_code = 0;
>>>      dc->condexec_mask = 0;
>>
>> Still no initialization of ->tmp_a64[]?
> 
> It doesn't need initialization, does it? It's a static array, and we
> fill it as new temps are requested. So all we need to do is clear
> the count so it starts "empty". Or have I missed something?

Well, barring bugs, no.

But I suggested clearing to TCGV_UNUSED for --enable-tcg-debug at minimum so as
to catch those.


r~


Reply via email to