<div dir="ltr"><div dir="ltr">Thanks, I will wait for a few days and then commit if there are no objections.<div><br></div><div>Nelson</div></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Thu, Feb 20, 2025 at 5:20 AM Charlie Jenkins <<a href="mailto:charlie@rivosinc.com">charlie@rivosinc.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Commit 6a04e8230707 ("RISC-V: Fix display of partial instructions")<br>
changed how objdump displays instructions. This enabled data of any size<br>
to be dumped. riscv_disassemble_data() only supports data of size 1,<br>
2, 4, or 8 bytes and any other number would cause an abort.<br>
<br>
Instead of aborting when encountering one of the other values, dump the<br>
data as ".<N>byte <val>", similar to what is done for instructions with<br>
".insn <N>, <val>".<br>
<br>
Signed-off-by: Charlie Jenkins <<a href="mailto:charlie@rivosinc.com" target="_blank">charlie@rivosinc.com</a>><br>
---<br>
Changes in v3:<br>
- Move change to dis-partial-insn-word test case to first patch<br>
- Link to v2: <a href="https://lore.kernel.org/r/20250218-fix_gas_abort-v2-0-14457aa7387e@rivosinc.com" rel="noreferrer" target="_blank">https://lore.kernel.org/r/20250218-fix_gas_abort-v2-0-14457aa7387e@rivosinc.com</a><br>
<br>
Changes in v2:<br>
- Dump arbitrary data with ".<N>byte <val>"<br>
- Link to v1: <a href="https://lore.kernel.org/r/20250211-fix_gas_abort-v1-0-afd9730f9c51@rivosinc.com" rel="noreferrer" target="_blank">https://lore.kernel.org/r/20250211-fix_gas_abort-v1-0-afd9730f9c51@rivosinc.com</a><br>
<br>
---<br>
Charlie Jenkins (2):<br>
      RISC-V: Fix abort when displaying .dword<br>
      RISC-V: Add testcase for 6 byte instruction<br>
<br>
 gas/testsuite/gas/riscv/dis-partial-insn-dword.d | 12 ++++++++++++<br>
 gas/testsuite/gas/riscv/dis-partial-insn-dword.s |  2 ++<br>
 gas/testsuite/gas/riscv/dis-partial-insn-word.d  |  2 +-<br>
 opcodes/riscv-dis.c                              | 19 ++++++++++---------<br>
 4 files changed, 25 insertions(+), 10 deletions(-)<br>
---<br>
base-commit: f1ffed29357a30162506c0c895f2927c7c2a4aba<br>
change-id: 20250211-fix_gas_abort-6d1e28b4ad46<br>
-- <br>
- Charlie<br>
<br>
</blockquote></div></div>