[PATCH] dependency list for static libraries
Nick Clifton
nickc@redhat.com
Wed Nov 25 11:17:55 GMT 2020
Hi Howard,
>> This linker plugin works with both ld and gold.
> Ping: anything else needed for this, docs, etc.?
Sorry, sorry - my mind has been a mile away.
So - the code in the patch looks good. I am running regression tests
now but I am not expecting to encounter any problems. There are some
things missing however, the first of which is documentation as you
guessed:
* A entry in ld/NEWS describing the new plugin.
* Documentation in ld/ld.texi describing how to use the plugin,
preferably with examples.
* One or more new tests in linker's testsuite to check the
functionality of the plugin. Obviously these would have to be
native only tests, but there are places in the testsuite
where this is checked. (Eg ld/testsuite/ld-bootstrap/bootstrap.exp)
One other thing - is the plugin installed into the correct location
if "make install" is run ? (Ie: ${libdir}/bfd-plugins). I have not
actually checked this yet, so maybe it works, but I would have guessed
that some extra work would need to be done on ld/Makefile.am to manage
the installation.
Cheers
Nick
More information about the Binutils
mailing list