[PATCH] D43234: [ELF] - Fix case of using both --icf and --symbol-ordering-file together.
George Rimar via llvm-commits
llvm-commits at lists.llvm.org
Fri Feb 16 08:07:04 PST 2018
>That is, can't we just have
>
> int &Priority = SectionOrder[D->Section->Repl];
> Priority = std::min(Priority, SymbolOrder.lookup(D->getName()));
>
>Cheers,
>Rafael
Weird. Honestly is what I initially did and what did not work,
but I can't remember why and can't reproduce failture. Probably just did something wrong,
as this should work I believe now. Will update the patch with that change.
George.
More information about the llvm-commits
mailing list