From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 7896) id 9B22D3858028; Thu, 22 Sep 2022 07:05:54 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 9B22D3858028 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable From: Tsukasa OI To: bfd-cvs@sourceware.org Subject: [binutils-gdb] RISC-V: Remove "b" operand type from disassembler X-Act-Checkin: binutils-gdb X-Git-Author: Tsukasa OI X-Git-Refname: refs/heads/master X-Git-Oldrev: c1ecdee7e09681096229c177bb5fba8a538ce446 X-Git-Newrev: 0383bce6502271455804daa533c9d141e7c3fc98 Message-Id: <20220922070554.9B22D3858028@sourceware.org> Date: Thu, 22 Sep 2022 07:05:54 +0000 (GMT) X-BeenThere: binutils-cvs@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Binutils-cvs mailing list List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 22 Sep 2022 07:05:54 -0000 https://sourceware.org/git/gitweb.cgi?p=3Dbinutils-gdb.git;h=3D0383bce65022= 71455804daa533c9d141e7c3fc98 commit 0383bce6502271455804daa533c9d141e7c3fc98 Author: Tsukasa OI Date: Sun Aug 7 15:03:28 2022 +0900 RISC-V: Remove "b" operand type from disassembler =20 There are a few operand types not used by any RISC-V instructions. =20 - Cx - Vf - Ve - [ - ] - b =20 But most of them has a reasoning to keep them: =20 - Cx : Same as "Ct" except it has a constraint to have rd =3D=3D = rs2 (similar to "Cw"). Although it hasn't used, its role is c= lear enough to implement a new instruction with this operand ty= pe. - Vf, Ve : Used by vector AMO instructions (not ratified and real instructions are not upstreamed yet). - [, ] : Unused tokenization symbols. Reserving them is not harmful and a vendor may use this symbol for special purposes. =20 ... except "b". I could not have found any reference to this operand t= ype except it works like the "s" operand type. Historically, it seems... i= t's just unused from the beginning. Its role is not clear either. =20 On such cases, we should vacate this room for the new operand type with much clearer roles. =20 opcodes/ChangeLog: =20 * riscv-dis.c (print_insn_args): Remove 'b' operand type. Diff: --- opcodes/riscv-dis.c | 1 - 1 file changed, 1 deletion(-) diff --git a/opcodes/riscv-dis.c b/opcodes/riscv-dis.c index 7ae6e709290..99cebf37d9e 100644 --- a/opcodes/riscv-dis.c +++ b/opcodes/riscv-dis.c @@ -403,7 +403,6 @@ print_insn_args (const char *oparg, insn_t l, bfd_vma p= c, disassemble_info *info print (info->stream, dis_style_immediate, "0"); break; =20 - case 'b': case 's': if ((l & MASK_JALR) =3D=3D MATCH_JALR) maybe_print_address (pd, rs1, EXTRACT_ITYPE_IMM (l), 0);