[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