[PATCH] D33634: [ThinLTO] Reduce unnecessary map lookups during combined summary write

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 30 17:09:09 PDT 2017


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

LGTM



================
Comment at: lib/Bitcode/Writer/BitcodeWriter.cpp:381
 
   bool hasValueId(GlobalValue::GUID ValGUID) {
     const auto &VMI = GUIDToValueIdMap.find(ValGUID);
----------------
I think you can now remove this function as it is unused.


https://reviews.llvm.org/D33634





More information about the llvm-commits mailing list