On 12/4/19 7:53 AM, Alex Bennée wrote:
> 
> Richard Henderson <richard.hender...@linaro.org> writes:
> 
>> On 12/4/19 3:43 AM, Alex Bennée wrote:
> <snip>
>>>>  void gen_intermediate_code(CPUState *cpu, TranslationBlock *tb, int 
>>>> max_insns)
>>>>  {
>>>> -    DisasContext dc;
>>>> +    DisasContext dc = { };
>>>
>>> We seemed to have dropped an initialise here which seems unrelated.
>>
>> Added, not dropped.
> 
> But is it related to this patch or fixing another bug?

It is related to the patch.

We used to initialize all of the a32 and m32 fields in DisasContext by
assignment.  Now we only initialize either the a32 or m32 by assignment,
because the bits overlap in tbflags.  So zero out the other bits here.

I'll add this to the commit message.


r~

Reply via email to