Commit f6777964 authored by Reid Kleckner's avatar Reid Kleckner
Browse files

Use -fdebug-compilation-dir to form absolute paths in coverage mappings 

This allows users to explicitly request relative paths with
`-fdebug-compilation-dir .`.

Fixes PR43614

Reviewers: vsk, arphaman

Differential Revision: https://reviews.llvm.org/D68733

llvm-svn: 374266
parent b577efe4

clang/lib/CodeGen/CoverageMappingGen.cpp

+18 −7
Original line number Diff line number Diff line
@@ -1278,13 +1278,6 @@ std::string getCoverageSection(const CodeGenModule &CGM) { 
      CGM.getContext().getTargetInfo().getTriple().getObjectFormat());

}



std::string normalizeFilename(StringRef Filename) {

  llvm::SmallString<256> Path(Filename);

  llvm::sys::fs::make_absolute(Path);

  llvm::sys::path::remove_dots(Path, /*remove_dot_dot=*/true);

  return Path.str().str();

}



} // end anonymous namespace



static void dump(llvm::raw_ostream &OS, StringRef FunctionName,
@@ -1317,6 +1310,24 @@ static void dump(llvm::raw_ostream &OS, StringRef FunctionName, 
  }

}



CoverageMappingModuleGen::CoverageMappingModuleGen(

    CodeGenModule &CGM, CoverageSourceInfo &SourceInfo)

    : CGM(CGM), SourceInfo(SourceInfo), FunctionRecordTy(nullptr) {

  // Honor -fdebug-compilation-dir in paths in coverage data. Otherwise, use the

  // regular working directory when normalizing paths.

  if (!CGM.getCodeGenOpts().DebugCompilationDir.empty())

    CWD = CGM.getCodeGenOpts().DebugCompilationDir;

  else

    llvm::sys::fs::current_path(CWD);

}



std::string CoverageMappingModuleGen::normalizeFilename(StringRef Filename) {

  llvm::SmallString<256> Path(Filename);

  llvm::sys::fs::make_absolute(CWD, Path);

  llvm::sys::path::remove_dots(Path, /*remove_dot_dot=*/true);

  return Path.str().str();

}



void CoverageMappingModuleGen::addFunctionMappingRecord(

    llvm::GlobalVariable *NamePtr, StringRef NameValue, uint64_t FuncHash,

    const std::string &CoverageMapping, bool IsUsed) {

clang/lib/CodeGen/CoverageMappingGen.h

+6 −2
Original line number Diff line number Diff line
@@ -54,10 +54,14 @@ class CoverageMappingModuleGen { 
  std::vector<llvm::Constant *> FunctionNames;

  llvm::StructType *FunctionRecordTy;

  std::vector<std::string> CoverageMappings;

  SmallString<256> CWD;



  /// Make the filename absolute, remove dots, and normalize slashes to local

  /// path style.

  std::string normalizeFilename(StringRef Filename);



public:

  CoverageMappingModuleGen(CodeGenModule &CGM, CoverageSourceInfo &SourceInfo)

      : CGM(CGM), SourceInfo(SourceInfo), FunctionRecordTy(nullptr) {}

  CoverageMappingModuleGen(CodeGenModule &CGM, CoverageSourceInfo &SourceInfo);



  CoverageSourceInfo &getSourceInfo() const {

    return SourceInfo;

clang/test/CoverageMapping/debug-dir.cpp

0 → 100644
+16 −0
Original line number Diff line number Diff line 
// %s expands to an absolute path, so to test relative paths we need to create a

// clean directory, put the source there, and cd into it.

// RUN: rm -rf %t

// RUN: mkdir -p %t/foobar

// RUN: cd %t

// RUN: cp %s foobar/debug-dir.cpp



// RUN: %clang_cc1 -fprofile-instrument=clang -fcoverage-mapping -emit-llvm -main-file-name debug-dir.cpp foobar/debug-dir.cpp  -o - | FileCheck -check-prefix=ABSOLUTE %s

//

// ABSOLUTE: @__llvm_coverage_mapping = {{.*"\\01.*clang.*test.*CoverageMapping.*.*foobar.*debug-dir\.cpp}}



// RUN: %clang_cc1 -fprofile-instrument=clang -fcoverage-mapping -emit-llvm -main-file-name debug-dir.cpp foobar/debug-dir.cpp -fdebug-compilation-dir . -o - | FileCheck -check-prefix=RELATIVE %s

//

// RELATIVE: @__llvm_coverage_mapping = {{.*"\\01[^/]*foobar.*debug-dir\.cpp}}



void f1() {}