[PATCH] D48511: [ThinLTO] Add per-module indexes to combined index consistently

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 25 17:51:05 PDT 2018


pcc added inline comments.


================
Comment at: lib/Bitcode/Reader/BitcodeReader.cpp:4812
 ModuleSummaryIndex::ModuleInfo *
 ModuleSummaryIndexBitcodeReader::addThisModule() {
   return TheIndex.addModule(ModulePath, ModuleId);
----------------
Can we make this function return void now?


Repository:
  rL LLVM

https://reviews.llvm.org/D48511





More information about the llvm-commits mailing list