[PATCH] D55047: [ThinLTO] Import local variables from the same module as caller

[Eugene Leviant via Phabricator via llvm-commits]

evgeny777 added inline comments.


================
Comment at: lib/Transforms/IPO/FunctionImport.cpp:304
+      return GlobalValue::isLocalLinkage(RefSummary->linkage()) &&
+             VI.getSummaryList().size() > 1 &&
+             RefSummary->modulePath() != Summary.modulePath();
----------------
Why do you need to check for `VI.getSummaryList().size() > 1`? New test cases are passing without it.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55047/new/

https://reviews.llvm.org/D55047