[llvm] r213981 - coverage: remove empty mapping regions

Alex Lorenz arphaman at gmail.com
Fri Jul 25 15:22:24 PDT 2014


Author: arphaman
Date: Fri Jul 25 17:22:24 2014
New Revision: 213981

URL: http://llvm.org/viewvc/llvm-project?rev=213981&view=rev
Log:
coverage: remove empty mapping regions

This patch removes the empty coverage mapping regions.
Those regions were produced by clang's old mapping region generation 
algorithm, but the new algorithm doesn't generate them.

Modified:
    llvm/trunk/include/llvm/ProfileData/CoverageMapping.h
    llvm/trunk/lib/ProfileData/CoverageMappingReader.cpp
    llvm/trunk/lib/ProfileData/CoverageMappingWriter.cpp

Modified: llvm/trunk/include/llvm/ProfileData/CoverageMapping.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/ProfileData/CoverageMapping.h?rev=213981&r1=213980&r2=213981&view=diff
==============================================================================
--- llvm/trunk/include/llvm/ProfileData/CoverageMapping.h (original)
+++ llvm/trunk/include/llvm/ProfileData/CoverageMapping.h Fri Jul 25 17:22:24 2014
@@ -139,10 +139,6 @@ struct CounterMappingRegion {
     /// such as for a macro instantiation or #include file.
     ExpansionRegion,
 
-    /// \brief An EmptyRegion represents a source range without code,
-    /// but with a distinct counter.
-    EmptyRegion,
-
     /// \brief A SkippedRegion represents a source range with code that
     /// was skipped by a preprocessor or similar means.
     SkippedRegion

Modified: llvm/trunk/lib/ProfileData/CoverageMappingReader.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/ProfileData/CoverageMappingReader.cpp?rev=213981&r1=213980&r2=213981&view=diff
==============================================================================
--- llvm/trunk/lib/ProfileData/CoverageMappingReader.cpp (original)
+++ llvm/trunk/lib/ProfileData/CoverageMappingReader.cpp Fri Jul 25 17:22:24 2014
@@ -160,9 +160,6 @@ std::error_code RawCoverageMappingReader
         case CounterMappingRegion::CodeRegion:
           // Don't do anything when we have a code region with a zero counter.
           break;
-        case CounterMappingRegion::EmptyRegion:
-          Kind = CounterMappingRegion::EmptyRegion;
-          break;
         case CounterMappingRegion::SkippedRegion:
           Kind = CounterMappingRegion::SkippedRegion;
           break;

Modified: llvm/trunk/lib/ProfileData/CoverageMappingWriter.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/ProfileData/CoverageMappingWriter.cpp?rev=213981&r1=213980&r2=213981&view=diff
==============================================================================
--- llvm/trunk/lib/ProfileData/CoverageMappingWriter.cpp (original)
+++ llvm/trunk/lib/ProfileData/CoverageMappingWriter.cpp Fri Jul 25 17:22:24 2014
@@ -172,11 +172,6 @@ void CoverageMappingWriter::write(raw_os
       encodeULEB128(EncodedTagExpandedFileID, OS);
       break;
     }
-    case CounterMappingRegion::EmptyRegion:
-      encodeULEB128(unsigned(I.Kind)
-                        << Counter::EncodingCounterTagAndExpansionRegionTagBits,
-                    OS);
-      break;
     case CounterMappingRegion::SkippedRegion:
       assert(Count.isZero());
       encodeULEB128(unsigned(I.Kind)





More information about the llvm-commits mailing list