[PATCH] D34124: IR: Tweak the API around adding modules to the summary index.

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 12 16:41:48 PDT 2017


pcc created this revision.
Herald added a subscriber: hiraditya.

The current name (addModulePath) and return value
(ModulePathStringTableTy::iterator) is a little confusing. This
API adds a module, not just a path. And the iterator is basically
just an implementation detail of the summary index. Address
both of those issues by renaming to addModule and introducing a
ModuleSummaryIndex::Module type that the function returns.


https://reviews.llvm.org/D34124

Files:
  llvm/include/llvm/IR/ModuleSummaryIndex.h
  llvm/lib/Bitcode/Reader/BitcodeReader.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D34124.102262.patch
Type: text/x-patch
Size: 5714 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170612/33a7347f/attachment.bin>


More information about the llvm-commits mailing list