[PATCH] D27242: Bitcode: Introduce BitcodeModule::{has, get}Summary().

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 30 20:48:39 PST 2016


mehdi_amini accepted this revision.
mehdi_amini added a comment.
This revision is now accepted and ready to land.

LGTM.



================
Comment at: llvm/lib/Bitcode/Reader/BitcodeReader.cpp:6100
             return error("Invalid hash length " + Twine(Record.size()).str());
-          if (!TheIndex)
-            break;
----------------
If `TheIndex` is no longer optional, we should make it a reference.


https://reviews.llvm.org/D27242





More information about the llvm-commits mailing list