From: David Blaikie Date: Tue, 1 Dec 2015 00:48:39 +0000 (+0000) Subject: [llvm-dwp] Initial partial prototype X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=bb823837e04dc83f63c5a6c021aae05655b68dc9;p=oota-llvm.git [llvm-dwp] Initial partial prototype This just concatenates the common DWP sections without doing any of the fancy DWP things like: 1) update str_offsets 2) deduplicating strings 3) merging/creating cu/tu_index Patches for these will follow shortly. (also not sure about target triple/object file type for this tool - do I really need a whole triple just to write an object file that contains purely static/hardcoded bytes in each section? & I guess I should just pick it based on the first input, maybe, rather than hardcoding for now - but we only produce .dwo on ELF platforms with objcopy for now anyway) git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@254355 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/include/llvm/ADT/StringMap.h b/include/llvm/ADT/StringMap.h index 194235fac57..700bb9e10ef 100644 --- a/include/llvm/ADT/StringMap.h +++ b/include/llvm/ADT/StringMap.h @@ -232,6 +232,13 @@ public: : StringMapImpl(InitialSize, static_cast(sizeof(MapEntryTy))), Allocator(A) {} + StringMap(std::initializer_list> List) + : StringMapImpl(static_cast(sizeof(MapEntryTy))) { + for (const auto &P : List) { + insert(P); + } + } + StringMap(StringMap &&RHS) : StringMapImpl(std::move(RHS)), Allocator(std::move(RHS.Allocator)) {} diff --git a/include/llvm/ADT/StringSet.h b/include/llvm/ADT/StringSet.h index 3e0cc200b6d..08626dc7af8 100644 --- a/include/llvm/ADT/StringSet.h +++ b/include/llvm/ADT/StringSet.h @@ -23,6 +23,11 @@ namespace llvm { class StringSet : public llvm::StringMap { typedef llvm::StringMap base; public: + StringSet() = default; + StringSet(std::initializer_list S) { + for (StringRef X : S) + insert(X); + } std::pair insert(StringRef Key) { assert(!Key.empty()); diff --git a/test/tools/llvm-dwp/Inputs/simple/a.cpp b/test/tools/llvm-dwp/Inputs/simple/a.cpp new file mode 100644 index 00000000000..f85d105a99f --- /dev/null +++ b/test/tools/llvm-dwp/Inputs/simple/a.cpp @@ -0,0 +1,2 @@ +struct foo { }; +foo a; diff --git a/test/tools/llvm-dwp/Inputs/simple/a.dwo b/test/tools/llvm-dwp/Inputs/simple/a.dwo new file mode 100644 index 00000000000..7bdb2a7b9f8 Binary files /dev/null and b/test/tools/llvm-dwp/Inputs/simple/a.dwo differ diff --git a/test/tools/llvm-dwp/Inputs/simple/b.cpp b/test/tools/llvm-dwp/Inputs/simple/b.cpp new file mode 100644 index 00000000000..fedcc160f41 --- /dev/null +++ b/test/tools/llvm-dwp/Inputs/simple/b.cpp @@ -0,0 +1,3 @@ +struct bar { }; +void b(bar) { +} diff --git a/test/tools/llvm-dwp/Inputs/simple/b.dwo b/test/tools/llvm-dwp/Inputs/simple/b.dwo new file mode 100644 index 00000000000..f41243dc722 Binary files /dev/null and b/test/tools/llvm-dwp/Inputs/simple/b.dwo differ diff --git a/test/tools/llvm-dwp/X86/lit.local.cfg b/test/tools/llvm-dwp/X86/lit.local.cfg new file mode 100644 index 00000000000..05f8b38b334 --- /dev/null +++ b/test/tools/llvm-dwp/X86/lit.local.cfg @@ -0,0 +1,4 @@ +if not 'X86' in config.root.targets: + config.unsupported = True + +config.suffixes = ['.test', '.cpp', '.m', '.s'] diff --git a/test/tools/llvm-dwp/X86/simple.test b/test/tools/llvm-dwp/X86/simple.test new file mode 100644 index 00000000000..26215bdc8c9 --- /dev/null +++ b/test/tools/llvm-dwp/X86/simple.test @@ -0,0 +1,58 @@ +RUN: llvm-dwp %p/../Inputs/simple/a.dwo %p/../Inputs/simple/b.dwo -o %t +RUN: llvm-dwarfdump %t | FileCheck %s + +FIXME: For some reason, piping straight from llvm-dwp to llvm-dwarfdump doesn't behave well - looks like dwarfdump is reading/closes before dwp has finished. + +DWP from non-type-unit debug info for these two translation units: +a.cpp: + struct foo { }; + foo a; + +b.cpp: + struct bar { }; + void b(bar) { + } + +CHECK: .debug_abbrev.dwo contents: +CHECK: Abbrev table for offset: 0x00000000 +CHECK: DW_TAG_compile_unit +CHECK: DW_TAG_variable +CHECK: DW_TAG_structure_type +CHECK: Abbrev table for offset: 0x00000031 +CHECK: DW_TAG_compile_unit +CHECK: DW_TAG_structure_type +CHECK: DW_TAG_subprogram +CHECK: DW_TAG_formal_parameter + +CHECK: .debug_info.dwo contents: +CHECK: 0x00000000: Compile Unit: length = 0x00000025 version = 0x0004 abbr_offset = 0x0000 addr_size = 0x08 (next unit at 0x00000029) +CHECK: DW_TAG_compile_unit +CHECK: DW_AT_name {{.*}} "a.cpp" +CHECK: DW_TAG_variable +CHECK: DW_AT_name {{.*}} "a" +CHECK: DW_TAG_structure_type +CHECK: DW_AT_name {{.*}} "foo" + +FIXME: Using cu_index, identify that abbr_offset is 0x0031, not 0x0000 +CHECK: 0x00000029: Compile Unit: length = 0x00000031 version = 0x0004 abbr_offset = 0x0000 addr_size = 0x08 (next unit at 0x0000005e) +FIXME: Using cu_index, use strings based on the right str index offset +CHECK: DW_AT_name {{.*}} "a.cpp" +FIXME: Using cu_index to find the right abbrevs at abbr_offset, this abbrevation should actually be structure_type +CHECK: DW_TAG_variable + +CHECK: .debug_cu_index contents: +FIXME: Emit and verify the cu_index contents + +CHECK: .debug_str.dwo contents: +CHECK: "clang version +CHECK: 0x[[ACPP:.*]]: "a.cpp" +FIXME: Remove duplicates +CHECK: "clang version +CHECK: 0x[[BCPP:.*]]: "b.cpp" + +CHECK: .debug_str_offsets.dwo contents: +CHECK: : 00000000 +CHECK: : [[ACPP]] +CHECK: : 00000000 +FIXME: Update str offset indexes, this should be BCPP \/ +CHECK: : [[ACPP]] diff --git a/tools/llvm-dwp/llvm-dwp.cpp b/tools/llvm-dwp/llvm-dwp.cpp index b2f997621be..7f9f6678db0 100644 --- a/tools/llvm-dwp/llvm-dwp.cpp +++ b/tools/llvm-dwp/llvm-dwp.cpp @@ -1,2 +1,144 @@ -int main() { +#include "llvm/ADT/STLExtras.h" +#include "llvm/ADT/StringSet.h" +#include "llvm/CodeGen/AsmPrinter.h" +#include "llvm/MC/MCAsmInfo.h" +#include "llvm/MC/MCContext.h" +#include "llvm/MC/MCInstrInfo.h" +#include "llvm/MC/MCObjectFileInfo.h" +#include "llvm/MC/MCRegisterInfo.h" +#include "llvm/MC/MCSectionELF.h" +#include "llvm/MC/MCStreamer.h" +#include "llvm/Object/ObjectFile.h" +#include "llvm/Support/Options.h" +#include "llvm/Support/FileSystem.h" +#include "llvm/Support/MemoryBuffer.h" +#include "llvm/Support/TargetRegistry.h" +#include "llvm/Support/raw_ostream.h" +#include "llvm/Target/TargetMachine.h" +#include "llvm/Support/TargetSelect.h" +#include +#include +#include + +using namespace llvm; +using namespace cl; + +OptionCategory DwpCategory("Specific Options"); +static list InputFiles(Positional, OneOrMore, + desc(""), cat(DwpCategory)); + +static opt OutputFilename(Required, "o", desc("Specify the output file."), + value_desc("filename"), cat(DwpCategory)); + +static int error(const Twine &Error, const Twine &Context) { + errs() << Twine("while processing ") + Context + ":\n"; + errs() << Twine("error: ") + Error + "\n"; + return 1; +} + +static std::error_code writeSection(MCStreamer &Out, MCSection *OutSection, + const object::SectionRef &Sym) { + StringRef Contents; + if (auto Err = Sym.getContents(Contents)) + return Err; + Out.SwitchSection(OutSection); + Out.EmitBytes(Contents); + return std::error_code(); +} + +static std::error_code write(MCStreamer &Out, ArrayRef Inputs) { + for (const auto &Input : Inputs) { + auto ErrOrObj = object::ObjectFile::createObjectFile(Input); + if (!ErrOrObj) + return ErrOrObj.getError(); + const auto *Obj = ErrOrObj->getBinary(); + for (const auto &Section : Obj->sections()) { + const auto &MCOFI = *Out.getContext().getObjectFileInfo(); + static const StringMap KnownSections = { + {"debug_info.dwo", MCOFI.getDwarfInfoDWOSection()}, + {"debug_types.dwo", MCOFI.getDwarfTypesDWOSection()}, + {"debug_str_offsets.dwo", MCOFI.getDwarfStrOffDWOSection()}, + {"debug_str.dwo", MCOFI.getDwarfStrDWOSection()}, + {"debug_loc.dwo", MCOFI.getDwarfLocDWOSection()}, + {"debug_abbrev.dwo", MCOFI.getDwarfAbbrevDWOSection()}}; + StringRef Name; + if (std::error_code Err = Section.getName(Name)) + return Err; + if (MCSection *OutSection = + KnownSections.lookup(Name.substr(Name.find_first_not_of("._")))) + if (auto Err = writeSection(Out, OutSection, Section)) + return Err; + } + } + return std::error_code(); +} + +int main(int argc, char** argv) { + + ParseCommandLineOptions(argc, argv, "merge split dwarf (.dwo) files"); + + llvm::InitializeAllTargetInfos(); + llvm::InitializeAllTargetMCs(); + llvm::InitializeAllTargets(); + llvm::InitializeAllAsmPrinters(); + + std::string ErrorStr; + StringRef Context = "dwarf streamer init"; + + Triple TheTriple("x86_64-linux-gnu"); + + // Get the target. + const Target *TheTarget = + TargetRegistry::lookupTarget("", TheTriple, ErrorStr); + if (!TheTarget) + return error(ErrorStr, Context); + std::string TripleName = TheTriple.getTriple(); + + // Create all the MC Objects. + std::unique_ptr MRI(TheTarget->createMCRegInfo(TripleName)); + if (!MRI) + return error(Twine("no register info for target ") + TripleName, Context); + + std::unique_ptr MAI(TheTarget->createMCAsmInfo(*MRI, TripleName)); + if (!MAI) + return error("no asm info for target " + TripleName, Context); + + MCObjectFileInfo MOFI; + MCContext MC(MAI.get(), MRI.get(), &MOFI); + MOFI.InitMCObjectFileInfo(TheTriple, Reloc::Default, CodeModel::Default, + MC); + + auto MAB = TheTarget->createMCAsmBackend(*MRI, TripleName, ""); + if (!MAB) + return error("no asm backend for target " + TripleName, Context); + + std::unique_ptr MII(TheTarget->createMCInstrInfo()); + if (!MII) + return error("no instr info info for target " + TripleName, Context); + + std::unique_ptr MSTI( + TheTarget->createMCSubtargetInfo(TripleName, "", "")); + if (!MSTI) + return error("no subtarget info for target " + TripleName, Context); + + MCCodeEmitter *MCE = TheTarget->createMCCodeEmitter(*MII, *MRI, MC); + if (!MCE) + return error("no code emitter for target " + TripleName, Context); + + // Create the output file. + std::error_code EC; + raw_fd_ostream OutFile(OutputFilename, EC, sys::fs::F_None); + if (EC) + return error(Twine(OutputFilename) + ": " + EC.message(), Context); + + std::unique_ptr MS(TheTarget->createMCObjectStreamer( + TheTriple, MC, *MAB, OutFile, MCE, *MSTI, false, + /*DWARFMustBeAtTheEnd*/ false)); + if (!MS) + return error("no object streamer for target " + TripleName, Context); + + if (auto Err = write(*MS, InputFiles)) + return error(Err.message(), "Writing DWP file"); + + MS->Finish(); }