[PATCH] D26130: [ELF] - Implemented --symbol-ordering-file option.

[Rafael Espíndola via llvm-commits]

Actually, the patch in PR28018 fixes a stratup crash too.

Cheers,
Rafael


On 7 November 2016 at 09:49, George Rimar <grimar at accesssoftek.com> wrote:
>
>>> If lld mislinks firefox, we shouldn't silently ignore it and try to link with gold, but open a bug instead (and/or try to fix).
>>>
>>
>>
>>The issue is https://llvm.org/bugs/show_bug.cgi?id=28018
>>
>>I am OK with having an option to merge ro and rw sections with the
>>same name and make the result rw, but I am not convinced it should be
>>the default.
>>
>>Cheers,
>>Rafael
>
> Just in case: we are talking about 2 different issues with firefox.
> PR28018 indeed contains a patch to **link** it with LLD, but after that
> it still did not launch fine for me, solution described here:
> https://reviews.llvm.org/D26130#587974
>
> George.