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

[Teresa Johnson via Phabricator via llvm-commits]

tejohnson added inline comments.


================
Comment at: llvm/lib/Bitcode/Reader/BitcodeReader.cpp:3076
   TheModule = M;
+  MDLoader = MetadataLoader(Stream, *M, ValueList,
+                            [&](unsigned ID) { return getTypeByID(ID); });
----------------
mehdi_amini wrote:
> tejohnson wrote:
> > Why not create this within the BitcodeReader constructor instead? AFAICT we only ever invoke this right after we construct a BitcodeReader anyway (from within BitcodeModule::getModuleImpl).
> The `MetadataLoader` needs a `Module`, which we only have here.
Oh, of course, that makes sense.


https://reviews.llvm.org/D27646