[PATCH] D45798: LowerTypeTests: Propagate symver directives

[Vlad Tsyrklevich via Phabricator via llvm-commits]

vlad.tsyrklevich added inline comments.


================
Comment at: lib/Transforms/IPO/ThinLTOBitcodeWriter.cpp:361
+    Function *F = M.getFunction(Name);
+    if (!F || F->use_empty())
+      return;
----------------
@pcc This should probably be `F->hasAddressTaken()` instead of `F->use_empty()`, but I seem to recall that that would be actually be to restrictive for which functions we currently export--is that right?


Repository:
  rL LLVM

https://reviews.llvm.org/D45798