Re: [PATCH 18/19] bfd/RISC-V: move _bfd_riscv_elf_merge_private_bfd_data()

Project / Subsystem

binutils / bfd/risc-v

Date

2026-04-29

Proposer

Jiawei <jiawei@iscas.ac.cn>

Source type

public_inbox

Consensus

Proposed

Sentiment

/10

Technical tradeoffs

  • None apparent; this is a straightforward refactoring.

All attributes

project
binutils
subsystem
bfd/risc-v
patch_id
discussion_id
2e1cf135-ac03-4172-af5c-4ee32552845d@iscas.ac.cn
source_type
public_inbox
title
Re: [PATCH 18/19] bfd/RISC-V: move _bfd_riscv_elf_merge_private_bfd_data()
headline
bfd/RISC-V: move _bfd_riscv_elf_merge_private_bfd_data()
tldr
The ARCH_SIZE parameter is factored out of _bfd_riscv_elf_merge_private_bfd_data() to avoid building the code twice.
proposer
Jiawei <jiawei@iscas.ac.cn>
consensus
Proposed
outcome
ready
sentiment_score
technical_tradeoffs
  • None apparent; this is a straightforward refactoring.
series_id
binutils:bfd/risc-v: move _bfd_riscv_elf_merge_private_bfd_data()
series_role
reply
series_parts
[]
tags
  • binutils
  • bfd
  • risc-v
  • code duplication
  • refactoring
bugzilla_url
date
2026-04-29T00:00:00.000Z

Re: [PATCH 18/19] bfd/RISC-V: move _bfd_riscv_elf_merge_private_bfd_data()

This patch modifies the _bfd_riscv_elf_merge_private_bfd_data() function in the RISC-V backend of the Binary File Descriptor (BFD) library. The change refactors the code to accept ARCH_SIZE as a function parameter, preventing the function from being built twice with different ARCH_SIZE values, thus reducing code duplication.