[PATCH] D47842: [ThinLTO] Add string saver onto index for value names

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jun 24 20:59:57 PDT 2018


tejohnson updated this revision to Diff 152624.
tejohnson added a comment.

As discussed on https://reviews.llvm.org/D47905, make the allocator and saver Optional. Also revert
change to make the ModuleSummaryIndexWrapperPass a unique_ptr instead of
Optional, as the emplace() method can be used to avoid the issues my change
introduced there.


Repository:
  rL LLVM

https://reviews.llvm.org/D47842

Files:
  include/llvm/IR/ModuleSummaryIndex.h
  lib/Analysis/ModuleSummaryAnalysis.cpp
  lib/Bitcode/Reader/BitcodeReader.cpp
  test/ThinLTO/X86/autoupgrade.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D47842.152624.patch
Type: text/x-patch
Size: 3904 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180625/120baf9c/attachment.bin>


More information about the llvm-commits mailing list