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

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 6 13:26:39 PDT 2018


tejohnson added a comment.

I forgot to add, StringSaver is not copyable (due to BumpPtrAllocator reference), therefore this required changes to the Index value on the ModuleSummaryIndexWrapperPass to hold a unique_ptr instead of Optional, so that it could be initialized without a copy.


Repository:
  rL LLVM

https://reviews.llvm.org/D47842





More information about the llvm-commits mailing list