Re: [PATCH v2 01/16] RISC-V: rename operand descriptor O4

Project / Subsystem

binutils / risc-v

Date

2026-05-15

Proposer

Jiawei <jiawei@iscas.ac.cn>

Source type

public_inbox

Consensus

Ready to Land

Sentiment

/10

Technical tradeoffs

  • Improved code readability and maintainability at the cost of a simple renaming change.

All attributes

project
binutils
subsystem
risc-v
patch_id
discussion_id
971a7d74-1e80-4958-8942-d7ec75263087@iscas.ac.cn
source_type
public_inbox
title
Re: [PATCH v2 01/16] RISC-V: rename operand descriptor O4
headline
RISC-V: rename operand descriptor O4
tldr
Renames the RISC-V operand descriptor `O4` to `O7` for consistency, as it describes a 7-bit field.
proposer
Jiawei <jiawei@iscas.ac.cn>
consensus
Ready to Land
outcome
ready
sentiment_score
technical_tradeoffs
  • Improved code readability and maintainability at the cost of a simple renaming change.
series_id
binutils:risc-v: rename operand descriptor o4
series_role
reply
series_parts
  • {"title":"RISC-V: rename operand descriptor O4","url":"https://inbox.sourceware.org/binutils/25ba2a4c-a254-4276-b03a-aae99d5b9eb1@suse.com"}
tags
  • risc-v
  • assembler
  • opcode
  • operand descriptor
  • consistency
bugzilla_url
date
2026-05-15T00:00:00.000Z

Re: [PATCH v2 01/16] RISC-V: rename operand descriptor O4

This patch renames the RISC-V operand descriptor O4 to O7 to better reflect that it represents a 7-bit opcode field. The renaming improves consistency with other operand descriptors (O<n> and F<n>). This change simplifies subsequent code modifications. Reviewers have indicated that the patch looks good.