Re: [PATCH v35 01/13] target/avr: Add outward facing interfaces and core CPU logic

[Aleksandar Markovic]

On Friday, November 22, 2019, Richard Henderson <address@hidden> wrote:
> On 11/21/19 8:53 PM, Michael Rolnik wrote:
> > It seems to be a huge investment. this function should parse the
> > binary data as `decode_insn` does, so I suggest to modify decodetree
> > tool to make decoding information available to the instruction print
> > function.
> > what do you think?
>
> See target/openrisc/disas.c, which makes use of decodetree.
> It shouldn't be difficult to do something slimiar for avr.

I support Richard's suggested direction. Since you already implemenred core AVR decoder using decodetree, and you have this openrisc QEMU disassembler example, could you perhaps give a try to what Richard said, Michael?

Aleksandar

 

> r~