This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] RISC-V: Disable shared library support for embedded elf.


On Fri, Jan 5, 2018 at 2:31 PM, Jim Wilson <jimw@sifive.com> wrote:
> This was tested with riscv{32,64}-{elf,linux} targets on check-binutils,
> check-gas, and check-ld.  There were no changes to the linux results.  The
> elf ld results dropped by over 150 and the other testsuites had no change.

I goofed here.  I originally wrote a patch for the binutils-2.29
branch and carefully tested it.  I then had to rewrite it as the
check_shared_library_support routine was changed, and no longer has an
explicit list of non-shared targets.  I was a little less careful in
testing the rewrite.  I saw the failures drop by over 150 and assumed
it worked.  But in fact I have 4 regressions for the embedded elf
targets.

There is one riscv specific test using -shared unnecessarily which is
easy to fix.

There are two riscv specific tests using -shared deliberately, I need
to conditionalize them on check_shared_library_support which is easy.

There is one generic elf test for --eh-frame-hdr that is failing.
This is ld-elf/eh-frame-hdr.d, and this is a little more complicated.
The --eh-frame-hdr option only exists if shared libraries are
supported, but the test is being run unconditionally.  I see that it
has an explicit list of xfail targets which is incomplete and
unscalable.  It should be run from shared.exp instead of from elf.exp,
as shared.exp calls check_shared_library_support.  However, elf.exp
has a glob on *.d, which means the test needs to be renamed and then
fit into shared.exp somehow which may require a little rewriting.

Jim


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]