[PATCH] D16727: [Profiling] Write out sparse indexed profiles
David Li via llvm-commits
llvm-commits at lists.llvm.org
Fri Jan 29 10:22:12 PST 2016
davidxl added inline comments.
================
Comment at: unittests/ProfileData/CoverageMappingTest.cpp:291
@@ -282,1 +290,3 @@
+TEST_P(MaybeSparseCoverageMappingTest, strip_filename_prefix) {
+ InstrProfRecord Record("file1:func", 0x1234, {0});
ProfileWriter.addRecord(std::move(Record));
----------------
So why just this one change. For instance, the previous test can also be changed with {0, 0} counts to cover both the sparse and non sparse paths.
http://reviews.llvm.org/D16727
More information about the llvm-commits
mailing list