[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