On 17/02/26, Taylor Simpson wrote:
> We pass the Hexagon CPU definition to disassemble_hexagon.  This allows
> decode_packet to know if the opcodes are supported.
> 
> Note that we print valid instructions in a packet when one or more is
> invalid.  Rather than this
> 0x0002128c:  0x1eae4fec       {       <invalid>
> 0x00021290:  0x1c434c04               <invalid>
> 0x00021294:  0x1e03edf0               <invalid> }
> 
> We print this
> 0x0002128c:  0x1eae4fec       {       <invalid>
> 0x00021290:  0x1c434c04               V4.w = vadd(V12.w,V3.w)
> 0x00021294:  0x1e03edf0               V16 = V13 }
> 
> Co-authored-by: Matheus Tavares Bernardino 
> <[email protected]>
> Co-authored-by: Brian Cain <[email protected]>
> Signed-off-by: Taylor Simpson <[email protected]>
> ---
>  target/hexagon/cpu_bits.h  |  4 +++-
>  target/hexagon/printinsn.h |  3 ++-
>  disas/hexagon.c            |  3 ++-
>  target/hexagon/cpu.c       |  2 ++
>  target/hexagon/decode.c    | 25 +++++++++++++++++++++----
>  target/hexagon/printinsn.c |  9 +++++++--
>  6 files changed, 37 insertions(+), 9 deletions(-)

Reviewed-by: Anton Johansson <[email protected]>

Reply via email to