[PATCH] D17212: [ThinLTO] Support for call graph in per-module and combined summary.

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 28 19:53:38 PST 2016


joker.eph added inline comments.

================
Comment at: lib/Bitcode/Reader/BitcodeReader.cpp:5528
@@ -5481,2 +5527,3 @@
         TheIndex->addFunctionInfo(FunctionGlobalId, std::move(FuncInfo));
+        ValueIdToCallGraphGUIDMap[ValueID] = Function::getGUID(ValueName);
       }
----------------
Shouldn't this be `ValueIdToCallGraphGUIDMap[ValueID] = Function::getGUID(FunctionGlobalId);` ?



http://reviews.llvm.org/D17212





More information about the llvm-commits mailing list