Index: test/tools/llvm-dwp/X86/dwos_list_from_exec_simple.test =================================================================== --- test/tools/llvm-dwp/X86/dwos_list_from_exec_simple.test +++ test/tools/llvm-dwp/X86/dwos_list_from_exec_simple.test @@ -0,0 +1,126 @@ +FIXME: for the reasons explained below +(see the description of how the thest binaries were generated) +we have to do a few hacky things to make this test portable. +It would be wonderful to find a better and simpler approach. + +REQUIRES: shell + +RUN: rm -rf %t +RUN: mkdir %t +RUN: cd %t +RUN: cp %p/../Inputs/dwos_list_from_exec/a.dwo a.dwo +RUN: cp %p/../Inputs/dwos_list_from_exec/b.dwo b.dwo +RUN: cp %p/../Inputs/dwos_list_from_exec/c.dwo c.dwo +RUN: cp %p/../Inputs/dwos_list_from_exec/d.dwo d.dwo +RUN: cp %p/../Inputs/dwos_list_from_exec/e.dwo e.dwo +RUN: cp %p/../Inputs/dwos_list_from_exec/main main +RUN: cp %p/../Inputs/dwos_list_from_exec/libd.so libd.so +RUN: llvm-dwp c.dwo e.dwo -e main -e libd.so -o pkg.dwp +RUN: llvm-dwarfdump pkg.dwp | FileCheck %s + +Build commands for test data: + +mkdir && cd +clang++ -g -O0 -gsplit-dwarf a.cpp b.cpp -o main +clang++ -g -O0 -gsplit-dwarf -c c.cpp -o c.o +clang++ -g -O0 -gsplit-dwarf -fPIC -shared d.cpp -o libd.so +clang++ -g -O0 -gsplit-dwarf -c e.cpp -o e.o +Note, that at this stage the binaries main and libd.so +contain dwarf::DW_AT_comp_dir with the absolute path +to the , thus they are not suitable +for portable tests. To overcome this difficulty we apply +the "binary patching" script convert.py to main and libd.so. + +convert.py: + +#!/usr/bin/env python +import sys +# Read the binary data from stdin +content = sys.stdin.read() +# The right-hand side should be the actual value of +old_value="" +new_value="./" + "/" * (len(old_value) - 2) +assert(len(old_value) == len(new_value)) +# Replace the absolute path to the build directory +# with the string of the same length of the following form: "./////.../". +# Since the length has not changed this transformation doesn't break +# the binary format. +new_content = content.replace(old_value, new_value) +sys.stdout.write(new_content) + + +sources: +a.cpp: + void a() {} + +b.cpp: + void b() {} + int main() { + return 0; + } + +c.cpp: + void c() {} + +d.cpp: + void d() {} + +e.cpp: + void e() {} + +CHECK-LABEL: .debug_abbrev.dwo contents: + +CHECK-LABEL: Abbrev table for offset: +CHECK: DW_TAG_compile_unit +CHECK: DW_TAG_subprogram + +CHECK-LABEL: Abbrev table for offset: +CHECK: DW_TAG_compile_unit +CHECK: DW_TAG_subprogram + +CHECK-LABEL: Abbrev table for offset: +CHECK: DW_TAG_compile_unit +CHECK: DW_TAG_subprogram + +CHECK-LABEL: Abbrev table for offset: +CHECK: DW_TAG_compile_unit +CHECK: DW_TAG_subprogram + +CHECK-LABEL: Abbrev table for offset: +CHECK: DW_TAG_compile_unit +CHECK: DW_TAG_subprogram + +CHECK: .debug_info.dwo contents: +CHECK: [[AOFF:0x[0-9a-f]*]]: + +CHECK-LABEL: Compile Unit: length = {{.*}} version = 0x0004 +CHECK: DW_TAG_compile_unit +CHECK: DW_AT_name {{.*}} "c.cpp" +CHECK: DW_TAG_subprogram +CHECK: DW_AT_name {{.*}} "c" + +CHECK-LABEL: Compile Unit: length = {{.*}} version = 0x0004 +CHECK: DW_TAG_compile_unit +CHECK: DW_AT_name {{.*}} "e.cpp" +CHECK: DW_TAG_subprogram +CHECK: DW_AT_name {{.*}} "e" + +CHECK-LABEL: Compile Unit: length = {{.*}} version = 0x0004 +CHECK: DW_TAG_compile_unit +CHECK: DW_AT_name {{.*}} "a.cpp" +CHECK: DW_TAG_subprogram +CHECK: DW_AT_name {{.*}} "a" + +CHECK-LABEL: Compile Unit: length = {{.*}} version = 0x0004 +CHECK: DW_TAG_compile_unit +CHECK: DW_AT_name {{.*}} "b.cpp" +CHECK: DW_TAG_subprogram +CHECK: DW_AT_name {{.*}} "b" +CHECK: DW_TAG_subprogram +CHECK: DW_AT_name {{.*}} "main" + +CHECK-LABEL: Compile Unit: length = {{.*}} version = 0x0004 +CHECK: DW_TAG_compile_unit +CHECK: DW_AT_name {{.*}} "d.cpp" +CHECK: DW_TAG_subprogram +CHECK: DW_AT_name {{.*}} "d" Index: tools/llvm-dwp/llvm-dwp.cpp =================================================================== --- tools/llvm-dwp/llvm-dwp.cpp +++ tools/llvm-dwp/llvm-dwp.cpp @@ -17,6 +17,7 @@ #include "llvm/ADT/STLExtras.h" #include "llvm/ADT/StringSet.h" #include "llvm/CodeGen/AsmPrinter.h" +#include "llvm/DebugInfo/DWARF/DWARFContext.h" #include "llvm/DebugInfo/DWARF/DWARFFormValue.h" #include "llvm/DebugInfo/DWARF/DWARFUnitIndex.h" #include "llvm/MC/MCAsmInfo.h" @@ -36,6 +37,7 @@ #include "llvm/Support/MathExtras.h" #include "llvm/Support/MemoryBuffer.h" #include "llvm/Support/Options.h" +#include "llvm/Support/Path.h" #include "llvm/Support/TargetRegistry.h" #include "llvm/Support/TargetSelect.h" #include "llvm/Support/raw_ostream.h" @@ -49,9 +51,14 @@ using namespace cl; OptionCategory DwpCategory("Specific Options"); -static list InputFiles(Positional, OneOrMore, +static list InputFiles(Positional, ZeroOrMore, desc(""), cat(DwpCategory)); +static list ExecFilenames( + "e", ZeroOrMore, + desc("Specify the executable/library files to get the list of *.dwo from"), + value_desc("filename"), cat(DwpCategory)); + static opt OutputFilename(Required, "o", desc("Specify the output file."), value_desc("filename"), @@ -113,7 +120,7 @@ }; static Expected -getIndexedString(dwarf::Form Form, DataExtractor InfoData, +getIndexedString(dwarf::Form Form, DataExtractor InfoData, uint32_t &InfoOffset, StringRef StrOffsets, StringRef Str) { if (Form == dwarf::DW_FORM_string) return InfoData.getCStr(&InfoOffset); @@ -463,6 +470,35 @@ " and " + buildDWODescription(ID.Name, DWPName, ID.DWOName)); } +static Expected> +getDWOFilenames(StringRef ExecFilename) { + auto ErrOrObj = object::ObjectFile::createObjectFile(ExecFilename); + if (!ErrOrObj) + return ErrOrObj.takeError(); + + const ObjectFile &Obj = *ErrOrObj.get().getBinary(); + std::unique_ptr DWARFCtx = DWARFContext::create(Obj); + + SmallVector DWOPaths; + for (const auto &CU : DWARFCtx->compile_units()) { + const DWARFDie &Die = CU->getUnitDIE(); + std::string DWOName = dwarf::toString( + Die.find({dwarf::DW_AT_dwo_name, dwarf::DW_AT_GNU_dwo_name}), ""); + if (DWOName.empty()) + continue; + std::string DWOCompDir = + dwarf::toString(Die.find(dwarf::DW_AT_comp_dir), ""); + if (!DWOCompDir.empty()) { + SmallString<16> DWOPath; + sys::path::append(DWOPath, DWOCompDir, DWOName); + DWOPaths.emplace_back(DWOPath.data(), DWOPath.size()); + } else { + DWOPaths.push_back(std::move(DWOName)); + } + } + return std::move(DWOPaths); +} + static Error write(MCStreamer &Out, ArrayRef Inputs) { const auto &MCOFI = *Out.getContext().getObjectFileInfo(); MCSection *const StrSection = MCOFI.getDwarfStrDWOSection(); @@ -676,7 +712,19 @@ if (!MS) return error("no object streamer for target " + TripleName, Context); - if (auto Err = write(*MS, InputFiles)) { + std::vector DWOFilenames = InputFiles; + for (const auto &ExecFilename : ExecFilenames) { + auto DWOs = getDWOFilenames(ExecFilename); + if (!DWOs) { + logAllUnhandledErrors(DWOs.takeError(), errs(), "error: "); + return 1; + } + DWOFilenames.insert(DWOFilenames.end(), + std::make_move_iterator(DWOs->begin()), + std::make_move_iterator(DWOs->end())); + } + + if (auto Err = write(*MS, DWOFilenames)) { logAllUnhandledErrors(std::move(Err), errs(), "error: "); return 1; }