[PATCH] D40970: [ThinLTO] Remove unused (?) code from thinLTOInternalizeModule

Eugene Leviant via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 8 01:54:04 PST 2017


evgeny777 added inline comments.


================
Comment at: lib/Transforms/IPO/FunctionImport.cpp:647
-  // the current module.
-  StringSet<> AsmUndefinedRefs;
-  ModuleSymbolTable::CollectAsmSymbols(
----------------
The thinLTOInternalizeModule running in separate thread for each module being LTO'ed
So if we found some undefined symbol, which defined in another module what we can do now? That other 
module might have already been internalized (or is being internalized at the same moment on another CPU core).



Repository:
  rL LLVM

https://reviews.llvm.org/D40970





More information about the llvm-commits mailing list