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

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 14 14:35:09 PDT 2017


>>I preferred the version where parseSymbolVersion returned null. Doing two
>>scans over the name doesn't seem like a good idea.
>>
>>Rui, would you be OK with this patch but with parseSymbolVersion
>>returning a bool instead of having a isDefaultVersion? We can always try
>>to improve it afterwards.
>>
>>Cheers,
>>Rafael
>
>I can upload such version in hext hour or two, so we can choose one from 3.
>
>George.

It is D34221.

George.


More information about the llvm-commits mailing list