[PATCH] LD/testsuite: Remove $LDFLAGS reference from `run_ld_link_tests'

Project / Subsystem

binutils / ld/testsuite

Date

2026-07-21

Proposer

"Maciej W. Rozycki" <macro@orcam.me.uk>

Source type

public_inbox

Consensus

Proposed

Sentiment

9/10

Technical tradeoffs

  • Correctness of test suite execution.
  • Avoiding redundant linker flags.

All attributes

project
binutils
subsystem
ld/testsuite
patch_id
discussion_id
alpine.DEB.2.21.2607211442190.14485@angie.orcam.me.uk
source_type
public_inbox
title
[PATCH] LD/testsuite: Remove $LDFLAGS reference from `run_ld_link_tests'
headline
LD/testsuite: Remove $LDFLAGS reference from `run_ld_link_tests'
tldr
Patch corrects ld testsuite to avoid double-inclusion of LDFLAGS.
proposer
"Maciej W. Rozycki" <macro@orcam.me.uk>
consensus
Proposed
outcome
proposed
sentiment_score
9
technical_tradeoffs
  • Correctness of test suite execution.
  • Avoiding redundant linker flags.
series_id
series_role
standalone
series_parts
[]
tags
  • binutils
  • ld
  • testsuite
  • regression
  • patch
bugzilla_url
date
2026-07-21T00:00:00.000Z

[PATCH] LD/testsuite: Remove $LDFLAGS reference from `run_ld_link_tests'

This patch addresses an issue in the binutils linker test suite where the LDFLAGS variable was inadvertently included twice in linker invocations. It corrects a mistake introduced by previous commits and ensures that LDFLAGS are applied only once, preventing regressions and aligning test output with expectations. Testing on hppa64-hp-hpux11.23 showed no regressions.