[PATCH] PR ld/20828: Move symbol version processing ahead of GC symbol sweep

[Alan Modra]

On Thu, Feb 09, 2017 at 07:48:50PM +0000, Maciej W. Rozycki wrote:
> 1. Executable is being linked.
> 
> 2. Symbol has been defined locally.
> 
> 3. Symbol is being dynamically exported.
> 
> 4. Symbol will be garbage-collected.
> 
> however #4 cannot happen together with #1-#3 at once, because meeting 
> #1-#3 means the symbol cannot be garbage-collected.  IOW, if we get to 
> here, then either `->dynindx == -1' and we don't care or `->dynindx != -1' 
> and this won't change.  So it looks like most of my update can be dropped 
> and only the part below retained.
> 
>  Have I missed anything here?

I think you are correct, and this update makes the original patch OK
to apply.  I'm willing to approve it.

-- 
Alan Modra
Australia Development Lab, IBM