[PATCH] D35436: [ThinLTO] Ensure we always select the same function copy to import

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 17 12:31:31 PDT 2017


tejohnson added a comment.

Unfortunately I had to revert this in r308206. A ThinLTO bot was failing with a linker undefined reference error:
http://green.lab.llvm.org/green/job/clang-stage2-configure-Rthinlto_build/3663/

It seems unlikely that this heuristic change would directly cause that, but I don't have time to dig into it at the moment. Hopefully it is reproducible with lld or gold (this was ld64 on Darwin). I am not sure if I will have much time to spend looking into it this week, unfortunately.


Repository:
  rL LLVM

https://reviews.llvm.org/D35436





More information about the llvm-commits mailing list