Re: [PATCH] RISCV-V: Add new relocation type for global array accesses with non-constant indices

Project / Subsystem

binutils / riscv-v

Date

2026-07-09

Proposer

Jan Beulich <jbeulich@suse.com>

Source type

public_inbox

Consensus

Proposed

Sentiment

/10

All attributes

project
binutils
subsystem
riscv-v
patch_id
discussion_id
cd4a4dbd-af14-42ff-871a-25a3ec751d8a@suse.com
source_type
public_inbox
title
Re: [PATCH] RISCV-V: Add new relocation type for global array accesses with non-constant indices
headline
RISCV-V: Add new relocation type for global array accesses with non-constant indices
tldr
Comments on the patch submission regarding ChangeLog format and indentation.
proposer
Jan Beulich <jbeulich@suse.com>
consensus
Proposed
outcome
proposed
sentiment_score
technical_tradeoffs
[]
series_id
series_role
standalone
series_parts
[]
tags
  • RISC-V
  • relocations
  • linker
  • patching
  • ChangeLog
bugzilla_url
date
2026-07-09T00:00:00.000Z

Re: [PATCH] RISCV-V: Add new relocation type for global array accesses with non-constant indices

Jan Beulich provides feedback on a patch adding new RISC-V relocation types, specifically addressing issues with the ChangeLog format and inconsistent indentation. He points out that ChangeLog entries should now be part of commit messages, not directly patched, and that the patch uses spaces where tabs should be used for indentation. The original patch introduced three new relocation types and updated ELF support for RISC-V to handle them.