[PATCH] D35334: ThinLTO Minimized Bitcode File Size Reduction

[Haojie Wang via Phabricator via llvm-commits]

haojiewang updated this revision to Diff 107227.
haojiewang added a comment.

1. Format the code using clang-format.
2. Move GenerateHash, Hasher, addToStrtab from ModuleBitcodeWriterBase to ModuleBitcodeWriter since they are not really used in ThinLinkBitcodeWriter. Now ThinLinkBitcodeWriter just use StrtabBuilder.add() directly to write symbol names to Strtab.
3. Update splitAndWriteThinLTOBitcode so that only the necessary information will be written into ThinLTO parts.
4. Functionally remove writing a module with a precomputed module hash.
5. Update the test files. Remove the NODEBUG tests in split.ll and no-type-md.ll. Thin link bitcode file content check is moved into distributed-import.ll.


https://reviews.llvm.org/D35334

Files:
  include/llvm/Bitcode/BitcodeWriter.h
  lib/Bitcode/Writer/BitcodeWriter.cpp
  lib/Transforms/IPO/ThinLTOBitcodeWriter.cpp
  test/ThinLTO/X86/distributed_import.ll
  test/Transforms/ThinLTOBitcodeWriter/no-type-md.ll
  test/Transforms/ThinLTOBitcodeWriter/split.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D35334.107227.patch
Type: text/x-patch
Size: 21028 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170719/df2801f1/attachment.bin>