[PATCH v4 0/6] LoongArch: colored disassembly and readability tweaks

[mengqinggang]

Hi,

Whether these patches should consider copyright issues and add
a sign-off at the of the commit message?


在 2023/2/16 上午2:02, WANG Xuerui 写道:
> From: WANG Xuerui <git@xen0n.name>
>
> Hi,
>
> This series implements colored output for LoongArch disassembly, and
> some minor tweaks to the output so there is less clutter.
>
> Changes from v3:
>
> - Fixed ld test cases (make check-gas and make check-ld both pass on
>    x86_64 and loongarch64)
> - Branch target address is now correctly printed in comment style, plus
>    code simplification as suggested
>
> Changes from v2:
>
> - Fixed test cases
> - Added the fixed "LoongArch: support disassembling certain pseudo-
>    instructions" patch into this series
> - Fixed ".insn" in the last patch to say ".word" instead (MIPS muscle
>    memory strikes back hard)
> - Fixed some commit messages
> - Added mengqinggang to Cc list
>
> WANG Xuerui (6):
>    LoongArch: support disassembling certain pseudo-instructions
>    opcodes/loongarch: remove unused code
>    opcodes/loongarch: implement style support in the disassembler
>    opcodes/loongarch: style disassembled address offsets as such
>    opcodes/loongarch: do not print hex notation for signed immediates
>    opcodes/loongarch: print unrecognized insn words with the .word
>      directive
>
>   gas/config/tc-loongarch.c                     |   3 +-
>   gas/testsuite/gas/loongarch/imm_ins.d         |  84 ++++++-------
>   gas/testsuite/gas/loongarch/imm_ins_32.d      |  54 ++++-----
>   gas/testsuite/gas/loongarch/imm_op.d          |  44 +++----
>   gas/testsuite/gas/loongarch/jmp_op.d          |  44 +++----
>   gas/testsuite/gas/loongarch/li.d              |   8 +-
>   gas/testsuite/gas/loongarch/load_store_op.d   |  80 ++++++------
>   gas/testsuite/gas/loongarch/macro_op.d        |   4 +-
>   gas/testsuite/gas/loongarch/macro_op_32.d     |   4 +-
>   .../gas/loongarch/macro_op_large_abs.d        |  14 +--
>   .../gas/loongarch/macro_op_large_pc.d         |  14 +--
>   gas/testsuite/gas/loongarch/nop.d             |   2 +-
>   gas/testsuite/gas/loongarch/privilege_op.d    |   8 +-
>   gas/testsuite/gas/loongarch/raw-insn.d        |  11 ++
>   gas/testsuite/gas/loongarch/raw-insn.s        |   7 ++
>   gas/testsuite/gas/loongarch/reloc.d           |   2 +-
>   include/opcode/loongarch.h                    |   7 +-
>   ld/testsuite/ld-loongarch-elf/jmp_op.d        |  40 +++---
>   ld/testsuite/ld-loongarch-elf/macro_op.d      |  24 ++--
>   opcodes/disassemble.c                         |   5 +
>   opcodes/loongarch-dis.c                       | 114 ++++++++----------
>   opcodes/loongarch-opc.c                       |  73 ++++++-----
>   22 files changed, 333 insertions(+), 313 deletions(-)
>   create mode 100644 gas/testsuite/gas/loongarch/raw-insn.d
>   create mode 100644 gas/testsuite/gas/loongarch/raw-insn.s
>