This is the mail archive of the 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] COFF/PE: insert local symbols from foreign input objects into output executable

>>> On 25.10.11 at 11:03, Nick Clifton <> wrote:
> Hi Jan,
>> bfd/
>> 2011-09-28  Jan Beulich<>
>> 	* coffgen.c (coff_write_alien_symbol): Make public. Add 'struct
>> 	internal_syment *' parameter. Extend 'dummy' to an array with two
>> 	elements. Set n_numaux early. Handle BSF_FILE.
>> 	(coff_write_symbols): Pass NULL as new third argument to
>> 	coff_write_alien_symbol().
>> 	* cofflink.c (_bfd_coff_final_link): Don't use COFF-specific
>> 	obj_raw_syment_count() on non-COFF input BFD. Insert local symbols
>> 	from non-COFF input BFDs.
>> 	* libcoff-in.h (coff_write_alien_symbol): Declare.
>> 	* libcoff.h (coff_write_alien_symbol): Re-generate.
> Approved - please apply.
>> +  if (>strip != strip_all&&>discard != discard_all)
>> +    {
>> +      /* Add local symbols from foreign inputs.  */
>> +      for(sub = info->input_bfds; sub != NULL; sub = sub->link_next)
> I think that it would be worthwhile extending this comment to explain 
> why the local symbols are being preserved.

As it wasn't clear to me in what way this could be reasonably
extended (given the context, which I think already makes
things pretty clear), I committed this without adjustment to
the comment.


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