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

[Teresa Johnson via Phabricator via llvm-commits]

tejohnson created this revision.
tejohnson added reviewers: Vlad, pcc.
Herald added subscribers: steven_wu, inglorion, mehdi_amini.

Without this change we only add module paths to the combined index when
there is a module hash or at least one global value. Make this more
consistent by adding the module to the index whenever there is a summary
section, and it is a per-module summary (had a MODULE_CODE_SOURCE_FILENAME
record).

Since we will no longer add module paths lazily, add a new interface to get
the module info from the index that asserts it is already added.

Fixes PR37899.


Repository:
  rL LLVM

https://reviews.llvm.org/D48511

Files:
  include/llvm/IR/ModuleSummaryIndex.h
  lib/Bitcode/Reader/BitcodeReader.cpp
  test/LTO/Resolution/X86/mod-asm-used.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D48511.152559.patch
Type: text/x-patch
Size: 4566 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180623/152f74e6/attachment.bin>