[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