[PATCH] D27646: Refactor BitcodeReader: move Metadata and ValueId handling in their own class/file

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 9 17:07:44 PST 2016


mehdi_amini created this revision.
mehdi_amini added reviewers: pcc, tejohnson.
mehdi_amini added subscribers: dexonsmith, llvm-commits.
Herald added a subscriber: mgorny.

I'm planning on changing the way we load metadata to enable laziness.
I'm getting lost in this gigantic files, and gigantic class that is the bitcode
reader. This is a first toward splitting it in a few coarse components that
are more easily understandable.


https://reviews.llvm.org/D27646

Files:
  llvm/lib/Bitcode/Reader/BitcodeReader.cpp
  llvm/lib/Bitcode/Reader/CMakeLists.txt
  llvm/lib/Bitcode/Reader/MetadataLoader.cpp
  llvm/lib/Bitcode/Reader/MetadataLoader.h
  llvm/lib/Bitcode/Reader/ValueList.cpp
  llvm/lib/Bitcode/Reader/ValueList.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D27646.80983.patch
Type: text/x-patch
Size: 114551 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161210/9889b915/attachment.bin>


More information about the llvm-commits mailing list