Home Original page

[PATCH 00/59] Deduplicating CTF linker

Alan Modra amodra@gmail.com
Mon Jul 20 05:49:27 GMT 2020 
On Tue, Jul 14, 2020 at 10:31:27PM +0100, Nick Alcock via Binutils wrote:
> On 1 Jul 2020, Nick Alcock via Binutils said:
> 
> > So the deduplicating CTF linker is finally ready for review.  (Note: ready
> > for *review*, probably not yet quite ready for committing, since to speed
> > things up a bit I've posted before all non-Linux-arch tests are finished.
> > In particular, big-endian tests, Solaris tests, and tests on mingw and
> > cygwin will be done in parallel with this review.  Cross build-and-checks to
> > every supported arch, --enable-targets=all tests, and 64-bit and 32-bit
> > tests on x86-64 Linux and FreeBSD have been done already.)
> >
> > Please read and review as you wish -- most of these are libctf-only changes,
> > but I'd be very happy for review of those too.  There are only a few dozen
> > lines of non-testsuite changes outside libctf.
> 
> Ping? Patches 53, 54, 55, 56, 59 and possibly 57 and 58 are outside

These are OK.

> libctf and thus are awaiting review. (And I'd be happy with reviews for
> any of the others, too.)
> 
> I understand it's near release time and everyone's concentrating on
> that, and I'm quite happy to wait until after release: I'm just hoping
> this hasn't been entirely forgotten by everyone :)
> 
> Thanks!
> 
> 
> (There will be at least one more round of this series before pushing:
> there are a bunch of small fixes that fell out of cygwin and mingw
> testing. This series is long, so a repost of the whole thing with the
> fixes is probably not wanted: I'm not sure whether people would prefer
> fixups to the existing commits, a range-diff, or a simple repost of the
> corrected commits with changes notated in the commit log.)

Deltas from the previously posted patches would be good.  That should
be easy if you keep fixups separate, aiming to squash fixups before
committing.

-- 
Alan Modra
Australia Development Lab, IBM
More information about the Binutils mailing list