[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