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] PR ld/21382: Handle symbol defined in IR and referenced in DSO


On Wed, Apr 19, 2017 at 11:04:48AM -0700, H.J. Lu wrote:
> On Tue, Apr 18, 2017 at 5:48 PM, Alan Modra <amodra@gmail.com> wrote:
> > On Fri, Apr 14, 2017 at 08:44:56AM -0700, H.J. Lu wrote:
> >
> >> +++ b/bfd/elflink.c
> >> +           /* Preserve re_def_dynamic_ref so that this symbol will
> >
> > Comment typo.
> >
> >> +++ b/include/bfdlink.h
> >> +  /* Symbol is defined in an IR object and referenced in a dynamic
> >> +     object.  */
> >> +  unsigned int ir_def_dynamic_ref : 1;
> >
> > The comment here isn't completely true.  The flag will only be set if
> > the dynamic object is linked *after* the IR object, won't it?  I think
> > it is worth stating that, or fixing your plugin_notice change to also
> > cover the case where the IR object is linked after a dynamic object.
> >
> 
> Here is the updated patch.  I updated comments and renamed
> ir_def_dynamic_ref to dynamic_ref_after_ir_def.

OK.

-- 
Alan Modra
Australia Development Lab, IBM


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