[PATCH] D57203: [ThinLTO] Refine reachability check to fix compile time increase
Mehdi AMINI via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat Jan 26 17:10:04 PST 2019
mehdi_amini added inline comments.
================
Comment at: lib/Transforms/IPO/FunctionImport.cpp:830
+ visit(Index.getValueInfo(AS->getAliaseeGUID()));
+ continue;
+ }
----------------
I'm wondering about the `continue` here, the alias itself won't go through `Summary->setLive(true);` a few lines below, does it matter?
(I don't know if aliases can't have refs() as well, but it'll also skip the loop)
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