[Patch] skipping import libraries for performance reasons - direct auto-import of dll's
Nick Clifton
nickc@redhat.com
Tue Dec 17 02:04:00 GMT 2002
Hi Ralf,
> I've added a testcase, can anyone check this ?
I had a look over the test code itself. I do not currently have a
Cygwin environment available for testing, so I could not check to see
if the test worked but I do have a few comments on the harness code:
* Strictly speaking you should check for the presence of a C
compiler before using it to compile your tests and return UNTESTED
if one is not available. Of course since you are restricting your
tests to natives only, there must have been a C compiler around in
order to build the linker in the first place, but being thorough
never hurts.
* You should give yourself credit. ie you should mention that you
wrote the test case.
* Since the *-mingw* test is not currently supported, it should return
UNSUPPORTED or UNTESTED not FAIL. It is not the *-mingw*'s port's
fault that the test has not been written.
* Ideally you should follow the GNU coding conventions in your test
source files.
I am attaching a archive containing a version of your tests with the
above changes made. I would like however, to have it confirmed that
the tests do work on a real Cygwin target before I check them into the
binutils sources.
Cheers
Nick
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ld-testsuite-ld-auto-import.tgz
Type: application/x-gzip
Size: 1841 bytes
Desc: ld/testsuite/ld-auto-import
URL: <https://sourceware.org/pipermail/binutils/attachments/20021217/2793edb6/attachment.bin>
More information about the Binutils
mailing list