[PATCH] D43077: [ThinLTO] Import globals

[Teresa Johnson via Phabricator via llvm-commits]

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

LGTM with a request for a comment below.



================
Comment at: lib/Transforms/IPO/FunctionImport.cpp:259
+          !GlobalValue::isInterposableLinkage(RefSummary->linkage()) &&
+          RefSummary->refs().empty()) {
+        ImportList[RefSummary->modulePath()][VI.getGUID()] = 1;
----------------
Please add comment about why the restriction on having no outgoing refs.


https://reviews.llvm.org/D43077