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: gold patch committed: If no dynamic symbols, use normal symbols


"H.J. Lu" <hjl.tools@gmail.com> writes:

> On Fri, Jan 8, 2010 at 1:26 PM, Ian Lance Taylor <iant@google.com> wrote:
>> When the GNU linker sees a shared library with no dynamic symbols, it
>> uses the normal symbols instead. ÂThis turns out to be required on
>> some systems, as explained in slightly more detail in a comment in the
>> patch. ÂI committed this patch to gold to make it work the same way.
>>
>> Ian
>>
>>
>> 2010-01-08 ÂIan Lance Taylor Â<iant@google.com>
>>
>> Â Â Â ÂPR 11084
>> Â Â Â Â* dynobj.cc (Sized_dynobj::find_dynsym_sections): If there is no
>> Â Â Â Âdynamic symbol table, use the normal symbol table.
>> Â Â Â Â(Sized_dynobj::do_read_symbols): Remove assertion about type of
>> Â Â Â Âsymbol table.
>>
>
> This one failed to build with gcc 4.2 and 4.3:
>
> cc1plus: warnings being treated as errors
> /net/gnu-6/export/linux/src/binutils/binutils/gold/dynobj.cc: In member function
> âvoid gold::Sized_dynobj<size, big_endian>::find_dynsym_sections(const unsigned
> char*, unsigned int*, unsigned int*, unsigned int*, unsigned int*) [with int
> size = 64, bool big_endian = false]â:
> /net/gnu-6/export/linux/src/binutils/binutils/gold/dynobj.cc:118: warning: âpiâ
> may be used uninitialized in this function
> make[6]: *** [dynobj.o] Error 1

Yes, already fixed, sorry about that.

Ian


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