diff options
author | Vedant Kumar <vsk@apple.com> | 2017-09-15 23:00:00 +0000 |
---|---|---|
committer | Vedant Kumar <vsk@apple.com> | 2017-09-15 23:00:00 +0000 |
commit | b84e48447e4456bb29261b7380d9696bcf29f511 (patch) | |
tree | 1a6cc61e22fc4b1f1c44b21eacc18afcb4417583 /llvm/tools/llvm-cov/CoverageExporterJson.cpp | |
parent | eef6b2a5c97c139a447a2b1d45d48262be5239e3 (diff) | |
download | bcm5719-llvm-b84e48447e4456bb29261b7380d9696bcf29f511.tar.gz bcm5719-llvm-b84e48447e4456bb29261b7380d9696bcf29f511.zip |
[llvm-cov] Remove a redundant field. NFC.
The "NotCovered" fields in the region and line summary structs are
redundant. We should remove them to make the code clearer.
As a follow-up, the "NotCovered" entries should be removed from the
reports as well.
llvm-svn: 313415
Diffstat (limited to 'llvm/tools/llvm-cov/CoverageExporterJson.cpp')
-rw-r--r-- | llvm/tools/llvm-cov/CoverageExporterJson.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/llvm/tools/llvm-cov/CoverageExporterJson.cpp b/llvm/tools/llvm-cov/CoverageExporterJson.cpp index f891326ddfc..6ec6974f4c1 100644 --- a/llvm/tools/llvm-cov/CoverageExporterJson.cpp +++ b/llvm/tools/llvm-cov/CoverageExporterJson.cpp @@ -395,7 +395,9 @@ class CoverageExporterJson { emitDictStart(); emitDictElement("count", Summary.RegionCoverage.NumRegions); emitDictElement("covered", Summary.RegionCoverage.Covered); - emitDictElement("notcovered", Summary.RegionCoverage.NotCovered); + emitDictElement("notcovered", + Summary.RegionCoverage.NumRegions - + Summary.RegionCoverage.Covered); emitDictElement("percent", Summary.RegionCoverage.getPercentCovered()); // End Region Coverage Summary. emitDictEnd(); |