[PATCH] D18342: Use Blob to record MDString

[Mehdi AMINI via llvm-commits]

joker.eph created this revision.
joker.eph added a reviewer: tejohnson.
joker.eph added a subscriber: llvm-commits.

This makes encoding/decoding much simpler/faster. This is taking
a bit more space because of alignment over the current encoding
which is already fixed 8-bits.

- llvm-bcanalyzer reports on a RelWithDebInfo compile of
  DAGISelEmitter.cpp that the space occupied by the MDString goes
  from 12134612b to 12586813b (+3.7%). The total file goes from
  18513491b to 18965683b (+2.4%).
- MaterializeMetadata (during ThinLTO-link of llvm-tblgen) goes
  from 8.2s to 6.4s (-24.4%).

http://reviews.llvm.org/D18342

Files:
  include/llvm/Support/StreamingMemoryObject.h
  lib/Bitcode/Reader/BitcodeReader.cpp
  lib/Bitcode/Writer/BitcodeWriter.cpp
  test/Bitcode/invalid.test

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D18342.51251.patch
Type: text/x-patch
Size: 4417 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160322/091f2c0e/attachment.bin>