Re: [PATCH 05/21] target/xtensa: support copying registers up to 64 bits wide

[Max Filippov]

On Wed, Jul 8, 2020 at 9:14 AM Richard Henderson
<richard.henderson@linaro.org> wrote:
>
> On 7/6/20 4:47 PM, Max Filippov wrote:
> > +                if (arg_copy[i].arg->num_bits <= 32) {
> > +                    temp = tcg_temp_local_new_i32();
> > +                    tcg_gen_mov_i32(temp, arg_copy[i].arg->in);
> > +                } else if (arg_copy[i].arg->num_bits <= 64) {
> > +                    temp = tcg_temp_local_new_i64();
> > +                    tcg_gen_mov_i64(temp, arg_copy[i].arg->in);
>
> This shouldn't compile.
>
> You can't assign both TCGv_i32 and TCGv_i64 to the same variable.
>
> What's going on here?

temp is a void pointer, as well as OpcodeArg::in and OpcodeArg::out.

-- 
Thanks.
-- Max