davidxl added a comment.
Strictly speaking, this patch requires a version bump of the indexed format.
The profile reader also needs to adjust the FunctionHash computation (either
using 0 or simple function hash) based on the version of the profile data.
Check with Justin/vsk to see if it is important to keep backward compatability
for these simple functions in older profile data.
On the other hand, I wonder what is the real root cause of the problem. The
dummy function record does not have its 'own' profile counts, so
if (std::error_code EC = ProfileReader.getFunctionCounts(
Record.FunctionName, Record.FunctionHash, Counts)) {
call in CoverageMapping::load (..)
method should return the the counts of the used function even with zero
functionhash. What did I miss?
http://reviews.llvm.org/D20287
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
