[PATCH] D21883: ThinLTO: Do not take into account whether a definition has multiple copies when promoting.

Teresa Johnson via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 29 21:28:16 PDT 2016


tejohnson accepted this revision.
tejohnson added a comment.
This revision is now accepted and ready to land.

This looks like a nice simplification. I'm guessing the special handling for the single copy case was because it predated ThinLTO internalization, but Mehdi can probably say for sure.

Do we have a test case for the single copy non-exported case to ensure that we end up with the same internalized linkage with this change? If not, please add one.


http://reviews.llvm.org/D21883





More information about the llvm-commits mailing list