[PATCH] D57203: [ThinLTO] Refine reachability check to fix compile time increase

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 25 07:19:42 PST 2019


tejohnson marked an inline comment as done.
tejohnson added inline comments.


================
Comment at: lib/Transforms/IPO/FunctionImport.cpp:781
+    for (auto &S : VI.getSummaryList())
+      if (S->isLive())
+        return;
----------------
mehdi_amini wrote:
> Just a drive-by nit: you can just turn the `all_of` to `any_of` I believe
Sure I could do that. Note the above was the code here before the commit of D56117. Will change to any_of.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57203





More information about the llvm-commits mailing list