[PATCH] D146876: [ThinLTO] Only import for non-prevailing interposable global variables

[Florian Hahn via Phabricator via llvm-commits]

fhahn added inline comments.


================
Comment at: llvm/include/llvm/Transforms/IPO/FunctionImport.h:161
+    function_ref<bool(GlobalValue::GUID, const GlobalValueSummary *)>
+        isPrevailing,
+    const ModuleSummaryIndex &Index, FunctionImporter::ImportMapTy &ImportList);
----------------
nit: The naming here is inconsistent with the other arguments; it should probably be `IsPrevailing`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146876