ikudrin abandoned this revision.
ikudrin added a comment.
This change is not needed anymore because the whole issue was fixed in
http://reviews.llvm.org/D20286.
http://reviews.llvm.org/D20287
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
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
ikudrin added a comment.
Does anyone known, why we need dummy coverage mapping records for unused
functions? How are they used? Isn't it better to remove these dummy records to
prevent confusion with the real ones?
http://reviews.llvm.org/D20287
ikudrin added a comment.
The motivation sample (using llvm-cov with http://reviews.llvm.org/D20286
applied):
$ cat > sample.h << EOF
inline int sample_func(int A) {
return A;
}
EOF
$ cat > dummy.cpp << EOF
#include "sample.h"
EOF
$ cat > sample.cpp << EOF
#include