[PATCH] D33680: [ELF] - Resolve references properly when using .symver directive

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 9 05:13:45 PDT 2017


>We used to parse the names there, but parsing at the current location is
>better IMHO.
>
>Instead of resolve, can you just call SymbolTable<ELFT>::addRegular? The
>symbol with a default version is always a DefinedRegular, no?
>
>Cheers,
>Rafael

That looks correct for me. Will update patch in a few minutes, thanks for this hint !

George.


More information about the llvm-commits mailing list