diff options
author | Vedant Kumar <vsk@apple.com> | 2017-11-09 02:33:43 +0000 |
---|---|---|
committer | Vedant Kumar <vsk@apple.com> | 2017-11-09 02:33:43 +0000 |
commit | 43247f05421ba6e6f247592be6c2f39778796c6e (patch) | |
tree | 7a1043fdaf2a1e6c1bb7d6b9546a068d8dd6895d /llvm/test/tools/llvm-cov/Inputs | |
parent | f9a0d44eea28ada1adf75e5e63274055fccd65ab (diff) | |
download | bcm5719-llvm-43247f05421ba6e6f247592be6c2f39778796c6e.tar.gz bcm5719-llvm-43247f05421ba6e6f247592be6c2f39778796c6e.zip |
[Coverage] Use the wrapped segment when a line has entry segments
We've worked around bugs in the frontend by ignoring the count from
wrapped segments when a line has at least one region entry segment.
Those frontend bugs are now fixed, so it's time to regenerate the
checked-in covmapping files and remove the workaround.
llvm-svn: 317761
Diffstat (limited to 'llvm/test/tools/llvm-cov/Inputs')
-rw-r--r-- | llvm/test/tools/llvm-cov/Inputs/deferred-regions.covmapping | bin | 796 -> 900 bytes | |||
-rw-r--r-- | llvm/test/tools/llvm-cov/Inputs/highlightedRanges.covmapping | bin | 360 -> 336 bytes | |||
-rw-r--r-- | llvm/test/tools/llvm-cov/Inputs/highlightedRanges.json | 11 | ||||
-rw-r--r-- | llvm/test/tools/llvm-cov/Inputs/instrprof-comdat.h | 6 | ||||
-rw-r--r-- | llvm/test/tools/llvm-cov/Inputs/lineExecutionCounts.covmapping | bin | 172 -> 196 bytes | |||
-rw-r--r-- | llvm/test/tools/llvm-cov/Inputs/prefer_used_to_unused.covmapping | bin | 420 -> 612 bytes | |||
-rw-r--r-- | llvm/test/tools/llvm-cov/Inputs/regionMarkers.covmapping | bin | 208 -> 196 bytes | |||
-rw-r--r-- | llvm/test/tools/llvm-cov/Inputs/regionMarkers.json | 11 | ||||
-rw-r--r-- | llvm/test/tools/llvm-cov/Inputs/report.covmapping | bin | 219 -> 224 bytes | |||
-rw-r--r-- | llvm/test/tools/llvm-cov/Inputs/templateInstantiations.covmapping | bin | 248 -> 316 bytes |
10 files changed, 13 insertions, 15 deletions
diff --git a/llvm/test/tools/llvm-cov/Inputs/deferred-regions.covmapping b/llvm/test/tools/llvm-cov/Inputs/deferred-regions.covmapping Binary files differindex d16db88acbe..6d7919eb8fa 100644 --- a/llvm/test/tools/llvm-cov/Inputs/deferred-regions.covmapping +++ b/llvm/test/tools/llvm-cov/Inputs/deferred-regions.covmapping diff --git a/llvm/test/tools/llvm-cov/Inputs/highlightedRanges.covmapping b/llvm/test/tools/llvm-cov/Inputs/highlightedRanges.covmapping Binary files differindex 5c1b1d61120..5c3d67f24e1 100644 --- a/llvm/test/tools/llvm-cov/Inputs/highlightedRanges.covmapping +++ b/llvm/test/tools/llvm-cov/Inputs/highlightedRanges.covmapping diff --git a/llvm/test/tools/llvm-cov/Inputs/highlightedRanges.json b/llvm/test/tools/llvm-cov/Inputs/highlightedRanges.json index 2948e86e453..cff6ab3e686 100644 --- a/llvm/test/tools/llvm-cov/Inputs/highlightedRanges.json +++ b/llvm/test/tools/llvm-cov/Inputs/highlightedRanges.json @@ -12,9 +12,9 @@ // Verify the Summary Section for the first file // CHECK-SAME: "summary":{ -// CHECK-SAME: "lines":{"count":40,"covered":27,"percent":67}, +// CHECK-SAME: "lines":{"count":40,"covered":33,"percent":82}, // CHECK-SAME: "functions":{"count":4,"covered":4,"percent":100}, -// CHECK-SAME: "regions":{"count":19,"covered":11,"notcovered":8,"percent":57}}} +// CHECK-SAME: "regions":{"count":21,"covered":13,"notcovered":8,"percent":61}}} // Close Files Array // CHECK-SAME: ], @@ -45,10 +45,7 @@ // Full Export Summary // CHECK-SAME: "totals":{ -// CHECK-SAME: "lines":{"count":40,"covered":27,"percent":67}, +// CHECK-SAME: "lines":{"count":40,"covered":33,"percent":82}, // CHECK-SAME: "functions":{"count":4,"covered":4,"percent":100}, // CHECK-SAME: "instantiations":{"count":4,"covered":4,"percent":100}, -// CHECK-SAME: "regions":{"count":19,"covered":11,"notcovered":8,"percent":57}} - -// Close the export object, data array, and root object -// CHECK-SAME: }]} +// CHECK-SAME: "regions":{"count":21,"covered":13,"notcovered":8,"percent":61}}} diff --git a/llvm/test/tools/llvm-cov/Inputs/instrprof-comdat.h b/llvm/test/tools/llvm-cov/Inputs/instrprof-comdat.h index 5a6d72b248a..07941f9bb49 100644 --- a/llvm/test/tools/llvm-cov/Inputs/instrprof-comdat.h +++ b/llvm/test/tools/llvm-cov/Inputs/instrprof-comdat.h @@ -12,7 +12,7 @@ template <class T> T FOO<T>::DoIt(T ti) { // HEADER: [[@LINE]]| 2|template for (T I = 0; I < ti; I++) { // HEADER: [[@LINE]]| 22| for (T t += I; // HEADER: [[@LINE]]| 20| t += I; if (I > ti / 2) // HEADER: [[@LINE]]| 20| if (I > ti - t -= 1; // HEADER: [[@LINE]]| 8| t -= 1; + t -= 1; // HEADER: [[@LINE]]| 20| t -= 1; } // HEADER: [[@LINE]]| 20| } // HEADER: [[@LINE]]| 2| return t; // HEADER: [[@LINE]]| 2| return t; @@ -22,3 +22,7 @@ template <class T> T FOO<T>::DoIt(T ti) { // HEADER: [[@LINE]]| 2|template // compile a program with two calls to Foo<int>::DoIt(10) for each desired // architecture. Collect a raw profile from any one of these binaries, index // it, and check it in along with the executables. + +// Note: +// The line execution count on line 14 is incorrect because of a bug in the +// checked-in coverage mappings, which has subsequently been fixed in clang. diff --git a/llvm/test/tools/llvm-cov/Inputs/lineExecutionCounts.covmapping b/llvm/test/tools/llvm-cov/Inputs/lineExecutionCounts.covmapping Binary files differindex f70ea0528db..0b2458ae4fd 100644 --- a/llvm/test/tools/llvm-cov/Inputs/lineExecutionCounts.covmapping +++ b/llvm/test/tools/llvm-cov/Inputs/lineExecutionCounts.covmapping diff --git a/llvm/test/tools/llvm-cov/Inputs/prefer_used_to_unused.covmapping b/llvm/test/tools/llvm-cov/Inputs/prefer_used_to_unused.covmapping Binary files differindex 4722129fc18..134067e9194 100644 --- a/llvm/test/tools/llvm-cov/Inputs/prefer_used_to_unused.covmapping +++ b/llvm/test/tools/llvm-cov/Inputs/prefer_used_to_unused.covmapping diff --git a/llvm/test/tools/llvm-cov/Inputs/regionMarkers.covmapping b/llvm/test/tools/llvm-cov/Inputs/regionMarkers.covmapping Binary files differindex 1c9b5dd45c7..af920c2d169 100644 --- a/llvm/test/tools/llvm-cov/Inputs/regionMarkers.covmapping +++ b/llvm/test/tools/llvm-cov/Inputs/regionMarkers.covmapping diff --git a/llvm/test/tools/llvm-cov/Inputs/regionMarkers.json b/llvm/test/tools/llvm-cov/Inputs/regionMarkers.json index 0589f7b6626..c9964338947 100644 --- a/llvm/test/tools/llvm-cov/Inputs/regionMarkers.json +++ b/llvm/test/tools/llvm-cov/Inputs/regionMarkers.json @@ -12,9 +12,9 @@ // Verify the Summary Section for the first file // CHECK-SAME: "summary":{ -// CHECK-SAME: "lines":{"count":21,"covered":18,"percent":85}, +// CHECK-SAME: "lines":{"count":21,"covered":19,"percent":90}, // CHECK-SAME: "functions":{"count":1,"covered":1,"percent":100}, -// CHECK-SAME: "regions":{"count":10,"covered":7,"notcovered":3,"percent":70}} +// CHECK-SAME: "regions":{"count":11,"covered":8,"notcovered":3,"percent":72}} // Close Files Array // CHECK-SAME: ], @@ -29,10 +29,7 @@ // Full Export Summary // CHECK-SAME: "totals":{ -// CHECK-SAME: "lines":{"count":21,"covered":18,"percent":85}, +// CHECK-SAME: "lines":{"count":21,"covered":19,"percent":90}, // CHECK-SAME: "functions":{"count":1,"covered":1,"percent":100}, // CHECK-SAME: "instantiations":{"count":1,"covered":1,"percent":100}, -// CHECK-SAME: "regions":{"count":10,"covered":7,"notcovered":3,"percent":70}} - -// Close the export object, data array, and root object -// CHECK-SAME: }]} +// CHECK-SAME: "regions":{"count":11,"covered":8,"notcovered":3,"percent":72}} diff --git a/llvm/test/tools/llvm-cov/Inputs/report.covmapping b/llvm/test/tools/llvm-cov/Inputs/report.covmapping Binary files differindex 5d0bfc11681..f9858ed272b 100644 --- a/llvm/test/tools/llvm-cov/Inputs/report.covmapping +++ b/llvm/test/tools/llvm-cov/Inputs/report.covmapping diff --git a/llvm/test/tools/llvm-cov/Inputs/templateInstantiations.covmapping b/llvm/test/tools/llvm-cov/Inputs/templateInstantiations.covmapping Binary files differindex 2dfc6cddbe7..b604e0604ad 100644 --- a/llvm/test/tools/llvm-cov/Inputs/templateInstantiations.covmapping +++ b/llvm/test/tools/llvm-cov/Inputs/templateInstantiations.covmapping |