[PATCH] D26016: BitcodeReader: Require clients to read the block info block at most once.

[Jordan Rose via llvm-commits]

jordan_rose added a comment.

BLOCKINFO is part of the container format <http://llvm.org/docs/BitCodeFormat.html>, not part of an LLVM module. I'm objecting to the idea that it's better to write one BLOCKINFO for several "data blobs" (LLVM module, Swift module, whatever) than to just stick the BLOCKINFO in each data blob and be done with it. Right now we have a completely concatenative format, as long as you separate out the fixed-size magic number at the start; this would break that.


https://reviews.llvm.org/D26016