-
Notifications
You must be signed in to change notification settings - Fork 1
riscv_long_insn_1
Tsukasa OI edited this page Nov 28, 2022
·
6 revisions
- Status: Merged for Binutils 2.40
- Branch:
riscv-long-insn-1
- Tracking PR: #91 (view Pull Request and Diff)
- Mailing List:
Commit bb996692bd96 ("RISC-V/gas: allow generating up to 176-bit instructions with .insn") was suddenly merged into master by Jan Beulich. Although this encoding is not considered frozen (see the section "Expanded Instruction-Length Encoding" in the ISA Manual), such attempt makes sense.
However, it was insufficient in some ways.
I quickly found a stack-based buffer overflow and fixed that.
Besides that, I found following issues related to long (assembler: .insn
-based, disassembler: unrecognized and printed as .byte
) instructions:
- Assembler: False "value conflicts with instruction length" error
- Assembler: DWARF line number information is missing with big numbers
- Disassembler: Instruction is trimmed with 64-bits