This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [patch][gold] Fix pr18327
- From: Rafael EspÃndola <rafael dot espindola at gmail dot com>
- To: Cary Coutant <ccoutant at gmail dot com>
- Cc: "H.J. Lu" <hjl dot tools at gmail dot com>, Binutils <binutils at sourceware dot org>, Ian Lance Taylor <iant at google dot com>
- Date: Mon, 27 Apr 2015 13:08:59 -0400
- Subject: Re: [patch][gold] Fix pr18327
- Authentication-results: sourceware.org; auth=none
- References: <CAG3jReJUyMKr1d9be0ah2=zKHLsht0RNjyNErnuSSpKZJLn3+A at mail dot gmail dot com> <CAJimCsHkrc_N6k5scG+4XPq6YLkaNz9S+d0Yfu2=G3NeJXLkBA at mail dot gmail dot com> <CAJimCsEU9oC0jx+EifQPbVutLZYZY19O2hNqS5uUOGry4qtp9w at mail dot gmail dot com> <CAG3jReKGoE+NwSxQC7VWAU9YMsS4NT9vbCqw_NYD+rszNH_aKg at mail dot gmail dot com> <CAMe9rOojvYa3sWC=dsEn-Ymefk047YXOBaHVm7T0PgsNCzB+UQ at mail dot gmail dot com> <CAG3jReLk_TYYx+RD-ibJkBAE9GchQXGczRe3jQpjBfUJPSymDg at mail dot gmail dot com> <CAJimCsFLGyR0y+3nQv-Z4EYUu21QwnO94c_ekozM9N011X9UZQ at mail dot gmail dot com>
On 27 April 2015 at 13:07, Cary Coutant <ccoutant@gmail.com> wrote:
>> 2015-04-27 Rafael Ãvila de EspÃndola <rafael.espindola@gmail.com>
>>
>> PR gold/18327
>> * output.cc (Output_section::is_input_address_mapped): Assume a missing
>> entry is mapped.
>> * testsuite/Makefile.am: Add the eh_test test.
>> * testsuite/Makefile.in: Regenerate.
>> * testsuite/eh_test_a.cc: New test.
>> * testsuite/eh_test_b.cc: New test.
>
> +eh_test: eh_test_a.o eh_test_b.o gcctestdir/ld
> + $(CXXLINK_S) -Bgcctestdir/ eh_test_a.o eh_test_b.o
>
> I don't think you need to statically link in the runtime libraries.
> $(CXXLINK) should be fine here.
It is the other way, I need libstdc++ to be shared for the test to
fail on master, and:
# Strip out -static-libgcc and -static-libstdc++ options, for tests
# that must have these libraries linked dynamically. The -shared-libgcc
# option does not work correctly, and there is no -shared-libstdc++ option.
# (See GCC PR 55781 and PR 55782.)
CXXLINK_S = `echo $(CXXLINK1) | sed -e 's/-static-lib\\(gcc\\|stdc++\\)//g'`
OK?
Cheers,
Rafael