public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* Two unreviewed ARM patches
@ 2011-01-10 10:23 Richard Sandiford
  2011-01-17  9:44 ` PING^2: " Richard Sandiford
  0 siblings, 1 reply; 3+ messages in thread
From: Richard Sandiford @ 2011-01-10 10:23 UTC (permalink / raw)
  To: binutils; +Cc: Nick Clifton, Richard Earnshaw, Paul Brook

Just wanted to ping these two ARM patches:

http://sourceware.org/ml/binutils/2010-12/msg00473.html
Fix undercounting of possibly-dynamic relocs in check_relocs.
(Ignore the bit at the end about a follow-on patch for PIEs;
the current behaviour is correct.)

http://sourceware.org/ml/binutils/2010-12/msg00461.html
Fix a case where possibly-dynamic relocs remain unswept.

Richard

^ permalink raw reply	[flat|nested] 3+ messages in thread

* PING^2: Two unreviewed ARM patches
  2011-01-10 10:23 Two unreviewed ARM patches Richard Sandiford
@ 2011-01-17  9:44 ` Richard Sandiford
  2011-01-17 15:22   ` Nick Clifton
  0 siblings, 1 reply; 3+ messages in thread
From: Richard Sandiford @ 2011-01-17  9:44 UTC (permalink / raw)
  To: binutils; +Cc: Nick Clifton, Richard Earnshaw, Paul Brook

PING^2

Richard Sandiford <richard.sandiford@linaro.org> writes:
> Just wanted to ping these two ARM patches:
>
> http://sourceware.org/ml/binutils/2010-12/msg00473.html
> Fix undercounting of possibly-dynamic relocs in check_relocs.
> (Ignore the bit at the end about a follow-on patch for PIEs;
> the current behaviour is correct.)
>
> http://sourceware.org/ml/binutils/2010-12/msg00461.html
> Fix a case where possibly-dynamic relocs remain unswept.
>
> Richard

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: PING^2: Two unreviewed ARM patches
  2011-01-17  9:44 ` PING^2: " Richard Sandiford
@ 2011-01-17 15:22   ` Nick Clifton
  0 siblings, 0 replies; 3+ messages in thread
From: Nick Clifton @ 2011-01-17 15:22 UTC (permalink / raw)
  To: binutils, Richard Earnshaw, Paul Brook, richard.sandiford

Hi Richard,

   Sorry for the delay.

> bfd/
> 	* elf32-arm.c (elf32_arm_check_relocs): Check needs_plt rather than
> 	h->needs_plt when deciding whether to record a possible dynamic reloc.
>
> ld/testsuite/
> 	* ld-arm/arm-rel32.s, ld-arm/arm-rel32.d: New testcase.
> 	* ld-arm/arm-elf.exp: Run it.
>

Approved - please apply.

> bfd/
> 	* elf32-arm.c (elf32_arm_gc_sweep_hook): Remove all registered
> 	dynamic relocs for the removed section.
>
> ld/testsuite/
> 	* ld-arm/gc-thumb-lib.s, ld-arm/gc-thumb.s,
> 	ld-arm/gc-thumb.d: New test.
> 	* ld-arm/arm-elf.exp: Run it.

Also approved - please apply.

Cheers
   Nick


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2011-01-17 15:22 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-01-10 10:23 Two unreviewed ARM patches Richard Sandiford
2011-01-17  9:44 ` PING^2: " Richard Sandiford
2011-01-17 15:22   ` Nick Clifton

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).