[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 20 13:08:08 PDT 2018


tejohnson marked an inline comment as done.
tejohnson added inline comments.


================
Comment at: lib/Bitcode/Reader/BitcodeReader.cpp:4840
+          ValueGUID,
+          UseStrtab ? ValueName : TheIndex.saveString(ValueName.str())),
       OriginalNameID);
----------------
evgeny777 wrote:
> Probably reflect the purpose of this in comment above? Otherwise it's misleading.
Updated comment to reflect the new behavior.


Repository:
  rL LLVM

https://reviews.llvm.org/D47842





More information about the llvm-commits mailing list