[PATCH] D21018: [ELF] - Basic versioned symbols support implemented.

[Rafael Espíndola via llvm-commits]

On 16 June 2016 at 09:25, George Rimar <grimar at accesssoftek.com> wrote:
>>rafael added inline comments.
>>
>>================
>>Comment at: ELF/SymbolListFile.cpp:80
>>@@ +79,3 @@
>>+private:
>>+  void parseUnversionedScript();
>>+  void parseVersionedScript();
>>----------------
>>Do we really need to duplicate the parsing? We should be able to have a single parser and take different actions based on the name, no?
>
> That was done in that way in initial version of the patch (please see diff 1).
> And Rui requested to separate this during review.

I think the request was just for the semantic handling, no?

Cheers,
Rafael