From b1bdd69c7b355e9d07392223ec41408e121760b7 Mon Sep 17 00:00:00 2001 From: Zachary Turner Date: Wed, 31 May 2017 04:17:13 +0000 Subject: [PATCH] [ObjectYAML] Split CodeViewYAML into 3 pieces. The code was a mess and disorganized due to the sheer amount of it being in one file. So I'm splitting this into three files. One for CodeView types, one for CodeView symbols, and one for CodeView debug subsections. NFC. llvm-svn: 304278 --- ...ViewYAML.h => CodeViewYAMLDebugSections.h} | 47 +- include/llvm/ObjectYAML/CodeViewYAMLSymbols.h | 41 ++ include/llvm/ObjectYAML/CodeViewYAMLTypes.h | 48 ++ lib/ObjectYAML/CMakeLists.txt | 4 +- lib/ObjectYAML/CodeViewYAMLDebugSections.cpp | 127 ++++ lib/ObjectYAML/CodeViewYAMLSymbols.cpp | 496 +++++++++++++++ ...CodeViewYAML.cpp => CodeViewYAMLTypes.cpp} | 566 +----------------- tools/llvm-pdbdump/PdbYaml.cpp | 1 + tools/llvm-pdbdump/PdbYaml.h | 4 +- tools/llvm-pdbdump/YAMLOutputStyle.cpp | 1 - tools/llvm-pdbdump/llvm-pdbdump.cpp | 1 - 11 files changed, 728 insertions(+), 608 deletions(-) rename include/llvm/ObjectYAML/{CodeViewYAML.h => CodeViewYAMLDebugSections.h} (54%) create mode 100644 include/llvm/ObjectYAML/CodeViewYAMLSymbols.h create mode 100644 include/llvm/ObjectYAML/CodeViewYAMLTypes.h create mode 100644 lib/ObjectYAML/CodeViewYAMLDebugSections.cpp create mode 100644 lib/ObjectYAML/CodeViewYAMLSymbols.cpp rename lib/ObjectYAML/{CodeViewYAML.cpp => CodeViewYAMLTypes.cpp} (57%) diff --git a/include/llvm/ObjectYAML/CodeViewYAML.h b/include/llvm/ObjectYAML/CodeViewYAMLDebugSections.h similarity index 54% rename from include/llvm/ObjectYAML/CodeViewYAML.h rename to include/llvm/ObjectYAML/CodeViewYAMLDebugSections.h index 16a58ba1060..6ddae2e2b41 100644 --- a/include/llvm/ObjectYAML/CodeViewYAML.h +++ b/include/llvm/ObjectYAML/CodeViewYAMLDebugSections.h @@ -1,4 +1,4 @@ -//===- CodeViewYAML.cpp - CodeView YAMLIO implementation ------------------===// +//===- CodeViewYAMLDebugSections.h - CodeView YAMLIO debug sections -------===// // // The LLVM Compiler Infrastructure // @@ -12,33 +12,17 @@ // //===----------------------------------------------------------------------===// -#ifndef LLVM_OBJECTYAML_CODEVIEWYAML_H -#define LLVM_OBJECTYAML_CODEVIEWYAML_H +#ifndef LLVM_OBJECTYAML_CODEVIEWYAMLDEBUGSECTIONS_H +#define LLVM_OBJECTYAML_CODEVIEWYAMLDEBUGSECTIONS_H #include "llvm/DebugInfo/CodeView/CodeView.h" #include "llvm/DebugInfo/CodeView/DebugSubsection.h" -#include "llvm/DebugInfo/CodeView/SymbolDeserializer.h" -#include "llvm/DebugInfo/CodeView/SymbolRecord.h" -#include "llvm/DebugInfo/CodeView/SymbolSerializer.h" -#include "llvm/DebugInfo/CodeView/TypeDeserializer.h" -#include "llvm/DebugInfo/CodeView/TypeRecord.h" -#include "llvm/DebugInfo/CodeView/TypeSerializer.h" -#include "llvm/DebugInfo/CodeView/TypeTableBuilder.h" #include "llvm/ObjectYAML/YAML.h" namespace llvm { namespace CodeViewYAML { namespace detail { struct C13FragmentBase; -struct LeafRecordBase; -struct MemberRecordBase; -struct SymbolRecordBase; -} - -namespace detail { -struct MemberRecordBase; -struct LeafRecordBase; -struct SymbolRecordBase; } struct SourceLineEntry { @@ -96,37 +80,12 @@ struct SourceFileInfo { std::vector Inlinees; }; -struct MemberRecord { - std::shared_ptr Member; -}; - -struct LeafRecord { - std::shared_ptr Leaf; - - codeview::CVType toCodeViewRecord(BumpPtrAllocator &Allocator) const; - static Expected fromCodeViewRecord(codeview::CVType Type); -}; - -struct SymbolRecord { - std::shared_ptr Symbol; - - codeview::CVSymbol toCodeViewSymbol(BumpPtrAllocator &Allocator) const; - static Expected fromCodeViewSymbol(codeview::CVSymbol Symbol); -}; - struct C13DebugSection { std::vector Fragments; }; } // namespace CodeViewYAML } // namespace llvm -LLVM_YAML_DECLARE_MAPPING_TRAITS(CodeViewYAML::LeafRecord) -LLVM_YAML_DECLARE_MAPPING_TRAITS(CodeViewYAML::MemberRecord) -LLVM_YAML_DECLARE_MAPPING_TRAITS(CodeViewYAML::SymbolRecord) LLVM_YAML_DECLARE_MAPPING_TRAITS(CodeViewYAML::SourceFileInfo) -LLVM_YAML_IS_SEQUENCE_VECTOR(CodeViewYAML::LeafRecord) -LLVM_YAML_IS_SEQUENCE_VECTOR(CodeViewYAML::MemberRecord) -LLVM_YAML_IS_SEQUENCE_VECTOR(CodeViewYAML::SymbolRecord) - #endif diff --git a/include/llvm/ObjectYAML/CodeViewYAMLSymbols.h b/include/llvm/ObjectYAML/CodeViewYAMLSymbols.h new file mode 100644 index 00000000000..ee4e2ac9d40 --- /dev/null +++ b/include/llvm/ObjectYAML/CodeViewYAMLSymbols.h @@ -0,0 +1,41 @@ +//===- CodeViewYAMLSymbols.h - CodeView YAMLIO Symbol implementation ------===// +// +// The LLVM Compiler Infrastructure +// +// This file is distributed under the University of Illinois Open Source +// License. See LICENSE.TXT for details. +// +//===----------------------------------------------------------------------===// +// +// This file defines classes for handling the YAML representation of CodeView +// Debug Info. +// +//===----------------------------------------------------------------------===// + +#ifndef LLVM_OBJECTYAML_CODEVIEWYAMLSYMBOLS_H +#define LLVM_OBJECTYAML_CODEVIEWYAMLSYMBOLS_H + +#include "llvm/DebugInfo/CodeView/CodeView.h" +#include "llvm/DebugInfo/CodeView/SymbolRecord.h" +#include "llvm/ObjectYAML/YAML.h" + +namespace llvm { +namespace CodeViewYAML { +namespace detail { +struct SymbolRecordBase; +} + +struct SymbolRecord { + std::shared_ptr Symbol; + + codeview::CVSymbol toCodeViewSymbol(BumpPtrAllocator &Allocator) const; + static Expected fromCodeViewSymbol(codeview::CVSymbol Symbol); +}; + +} // namespace CodeViewYAML +} // namespace llvm + +LLVM_YAML_DECLARE_MAPPING_TRAITS(CodeViewYAML::SymbolRecord) +LLVM_YAML_IS_SEQUENCE_VECTOR(CodeViewYAML::SymbolRecord) + +#endif diff --git a/include/llvm/ObjectYAML/CodeViewYAMLTypes.h b/include/llvm/ObjectYAML/CodeViewYAMLTypes.h new file mode 100644 index 00000000000..a57ada34a4f --- /dev/null +++ b/include/llvm/ObjectYAML/CodeViewYAMLTypes.h @@ -0,0 +1,48 @@ +//===- CodeViewYAMLTypes.h - CodeView YAMLIO Type Record implementation ---===// +// +// The LLVM Compiler Infrastructure +// +// This file is distributed under the University of Illinois Open Source +// License. See LICENSE.TXT for details. +// +//===----------------------------------------------------------------------===// +// +// This file defines classes for handling the YAML representation of CodeView +// Debug Info. +// +//===----------------------------------------------------------------------===// + +#ifndef LLVM_OBJECTYAML_CODEVIEWYAMLTYPES_H +#define LLVM_OBJECTYAML_CODEVIEWYAMLTYPES_H + +#include "llvm/DebugInfo/CodeView/CodeView.h" +#include "llvm/DebugInfo/CodeView/TypeRecord.h" +#include "llvm/ObjectYAML/YAML.h" + +namespace llvm { +namespace CodeViewYAML { +namespace detail { +struct LeafRecordBase; +struct MemberRecordBase; +} + +struct MemberRecord { + std::shared_ptr Member; +}; + +struct LeafRecord { + std::shared_ptr Leaf; + + codeview::CVType toCodeViewRecord(BumpPtrAllocator &Allocator) const; + static Expected fromCodeViewRecord(codeview::CVType Type); +}; +} // namespace CodeViewYAML +} // namespace llvm + +LLVM_YAML_DECLARE_MAPPING_TRAITS(CodeViewYAML::LeafRecord) +LLVM_YAML_DECLARE_MAPPING_TRAITS(CodeViewYAML::MemberRecord) + +LLVM_YAML_IS_SEQUENCE_VECTOR(CodeViewYAML::LeafRecord) +LLVM_YAML_IS_SEQUENCE_VECTOR(CodeViewYAML::MemberRecord) + +#endif diff --git a/lib/ObjectYAML/CMakeLists.txt b/lib/ObjectYAML/CMakeLists.txt index 237a409d70b..7af0b9c194e 100644 --- a/lib/ObjectYAML/CMakeLists.txt +++ b/lib/ObjectYAML/CMakeLists.txt @@ -1,5 +1,7 @@ add_llvm_library(LLVMObjectYAML - CodeViewYAML.cpp + CodeViewYAMLTypes.cpp + CodeViewYAMLSymbols.cpp + CodeViewYAMLDebugSections.cpp COFFYAML.cpp DWARFEmitter.cpp DWARFVisitor.cpp diff --git a/lib/ObjectYAML/CodeViewYAMLDebugSections.cpp b/lib/ObjectYAML/CodeViewYAMLDebugSections.cpp new file mode 100644 index 00000000000..f652ff57f30 --- /dev/null +++ b/lib/ObjectYAML/CodeViewYAMLDebugSections.cpp @@ -0,0 +1,127 @@ +//===- CodeViewYAMLDebugSections.cpp - CodeView YAMLIO debug sections -----===// +// +// The LLVM Compiler Infrastructure +// +// This file is distributed under the University of Illinois Open Source +// License. See LICENSE.TXT for details. +// +//===----------------------------------------------------------------------===// +// +// This file defines classes for handling the YAML representation of CodeView +// Debug Info. +// +//===----------------------------------------------------------------------===// + +#include "llvm/ObjectYAML/CodeViewYAMLDebugSections.h" + +#include "llvm/ADT/StringExtras.h" +#include "llvm/ADT/StringSwitch.h" +#include "llvm/DebugInfo/CodeView/CodeViewError.h" +#include "llvm/DebugInfo/CodeView/EnumTables.h" +#include "llvm/DebugInfo/CodeView/SymbolRecord.h" + +using namespace llvm; +using namespace llvm::codeview; +using namespace llvm::CodeViewYAML; +using namespace llvm::CodeViewYAML::detail; +using namespace llvm::yaml; + +LLVM_YAML_IS_SEQUENCE_VECTOR(SourceFileChecksumEntry) +LLVM_YAML_IS_SEQUENCE_VECTOR(SourceLineEntry) +LLVM_YAML_IS_SEQUENCE_VECTOR(SourceColumnEntry) +LLVM_YAML_IS_SEQUENCE_VECTOR(SourceLineBlock) +LLVM_YAML_IS_SEQUENCE_VECTOR(SourceLineInfo) +LLVM_YAML_IS_SEQUENCE_VECTOR(InlineeSite) +LLVM_YAML_IS_SEQUENCE_VECTOR(InlineeInfo) +LLVM_YAML_IS_SEQUENCE_VECTOR(StringRef) + +LLVM_YAML_DECLARE_SCALAR_TRAITS(HexFormattedString, false) +LLVM_YAML_DECLARE_ENUM_TRAITS(FileChecksumKind) +LLVM_YAML_DECLARE_BITSET_TRAITS(LineFlags) + +LLVM_YAML_DECLARE_MAPPING_TRAITS(CodeViewYAML::SourceLineEntry) +LLVM_YAML_DECLARE_MAPPING_TRAITS(CodeViewYAML::SourceColumnEntry) +LLVM_YAML_DECLARE_MAPPING_TRAITS(CodeViewYAML::SourceFileChecksumEntry) +LLVM_YAML_DECLARE_MAPPING_TRAITS(CodeViewYAML::SourceLineInfo) +LLVM_YAML_DECLARE_MAPPING_TRAITS(CodeViewYAML::SourceLineBlock) +LLVM_YAML_DECLARE_MAPPING_TRAITS(CodeViewYAML::InlineeInfo) +LLVM_YAML_DECLARE_MAPPING_TRAITS(CodeViewYAML::InlineeSite) + +void ScalarBitSetTraits::bitset(IO &io, LineFlags &Flags) { + io.bitSetCase(Flags, "HasColumnInfo", LF_HaveColumns); + io.enumFallback(Flags); +} + +void ScalarEnumerationTraits::enumeration( + IO &io, FileChecksumKind &Kind) { + io.enumCase(Kind, "None", FileChecksumKind::None); + io.enumCase(Kind, "MD5", FileChecksumKind::MD5); + io.enumCase(Kind, "SHA1", FileChecksumKind::SHA1); + io.enumCase(Kind, "SHA256", FileChecksumKind::SHA256); +} + +void ScalarTraits::output(const HexFormattedString &Value, + void *ctx, raw_ostream &Out) { + StringRef Bytes(reinterpret_cast(Value.Bytes.data()), + Value.Bytes.size()); + Out << toHex(Bytes); +} + +StringRef ScalarTraits::input(StringRef Scalar, void *ctxt, + HexFormattedString &Value) { + std::string H = fromHex(Scalar); + Value.Bytes.assign(H.begin(), H.end()); + return StringRef(); +} + +void MappingTraits::mapping(IO &IO, SourceLineEntry &Obj) { + IO.mapRequired("Offset", Obj.Offset); + IO.mapRequired("LineStart", Obj.LineStart); + IO.mapRequired("IsStatement", Obj.IsStatement); + IO.mapRequired("EndDelta", Obj.EndDelta); +} + +void MappingTraits::mapping(IO &IO, SourceColumnEntry &Obj) { + IO.mapRequired("StartColumn", Obj.StartColumn); + IO.mapRequired("EndColumn", Obj.EndColumn); +} + +void MappingTraits::mapping(IO &IO, SourceLineBlock &Obj) { + IO.mapRequired("FileName", Obj.FileName); + IO.mapRequired("Lines", Obj.Lines); + IO.mapRequired("Columns", Obj.Columns); +} + +void MappingTraits::mapping( + IO &IO, SourceFileChecksumEntry &Obj) { + IO.mapRequired("FileName", Obj.FileName); + IO.mapRequired("Kind", Obj.Kind); + IO.mapRequired("Checksum", Obj.ChecksumBytes); +} + +void MappingTraits::mapping(IO &IO, SourceLineInfo &Obj) { + IO.mapRequired("CodeSize", Obj.CodeSize); + + IO.mapRequired("Flags", Obj.Flags); + IO.mapRequired("RelocOffset", Obj.RelocOffset); + IO.mapRequired("RelocSegment", Obj.RelocSegment); + IO.mapRequired("Blocks", Obj.Blocks); +} + +void MappingTraits::mapping(IO &IO, SourceFileInfo &Obj) { + IO.mapOptional("Checksums", Obj.FileChecksums); + IO.mapOptional("Lines", Obj.LineFragments); + IO.mapOptional("InlineeLines", Obj.Inlinees); +} + +void MappingTraits::mapping(IO &IO, InlineeSite &Obj) { + IO.mapRequired("FileName", Obj.FileName); + IO.mapRequired("LineNum", Obj.SourceLineNum); + IO.mapRequired("Inlinee", Obj.Inlinee); + IO.mapOptional("ExtraFiles", Obj.ExtraFiles); +} + +void MappingTraits::mapping(IO &IO, InlineeInfo &Obj) { + IO.mapRequired("HasExtraFiles", Obj.HasExtraFiles); + IO.mapRequired("Sites", Obj.Sites); +} diff --git a/lib/ObjectYAML/CodeViewYAMLSymbols.cpp b/lib/ObjectYAML/CodeViewYAMLSymbols.cpp new file mode 100644 index 00000000000..6e8bb5c7372 --- /dev/null +++ b/lib/ObjectYAML/CodeViewYAMLSymbols.cpp @@ -0,0 +1,496 @@ +//===- CodeViewYAMLSymbols.cpp - CodeView YAMLIO Symbol implementation ----===// +// +// The LLVM Compiler Infrastructure +// +// This file is distributed under the University of Illinois Open Source +// License. See LICENSE.TXT for details. +// +//===----------------------------------------------------------------------===// +// +// This file defines classes for handling the YAML representation of CodeView +// Debug Info. +// +//===----------------------------------------------------------------------===// + +#include "llvm/ObjectYAML/CodeViewYAMLSymbols.h" +#include "llvm/ADT/StringExtras.h" +#include "llvm/ADT/StringSwitch.h" +#include "llvm/DebugInfo/CodeView/CVTypeVisitor.h" +#include "llvm/DebugInfo/CodeView/CodeViewError.h" +#include "llvm/DebugInfo/CodeView/EnumTables.h" +#include "llvm/DebugInfo/CodeView/SymbolDeserializer.h" +#include "llvm/DebugInfo/CodeView/SymbolSerializer.h" + +using namespace llvm; +using namespace llvm::codeview; +using namespace llvm::CodeViewYAML; +using namespace llvm::CodeViewYAML::detail; +using namespace llvm::yaml; + +LLVM_YAML_IS_SEQUENCE_VECTOR(StringRef) +LLVM_YAML_IS_FLOW_SEQUENCE_VECTOR(TypeIndex) + +// We only need to declare these, the definitions are in CodeViewYAMLTypes.cpp +LLVM_YAML_DECLARE_SCALAR_TRAITS(APSInt, false) +LLVM_YAML_DECLARE_SCALAR_TRAITS(TypeIndex, false) + +LLVM_YAML_DECLARE_ENUM_TRAITS(SymbolKind) + +LLVM_YAML_DECLARE_BITSET_TRAITS(CompileSym2Flags) +LLVM_YAML_DECLARE_BITSET_TRAITS(CompileSym3Flags) +LLVM_YAML_DECLARE_BITSET_TRAITS(ExportFlags) +LLVM_YAML_DECLARE_BITSET_TRAITS(LocalSymFlags) +LLVM_YAML_DECLARE_BITSET_TRAITS(ProcSymFlags) +LLVM_YAML_DECLARE_BITSET_TRAITS(FrameProcedureOptions) +LLVM_YAML_DECLARE_ENUM_TRAITS(CPUType) +LLVM_YAML_DECLARE_ENUM_TRAITS(RegisterId) +LLVM_YAML_DECLARE_ENUM_TRAITS(TrampolineType) +LLVM_YAML_DECLARE_ENUM_TRAITS(ThunkOrdinal) + +void ScalarEnumerationTraits::enumeration(IO &io, + SymbolKind &Value) { + auto SymbolNames = getSymbolTypeNames(); + for (const auto &E : SymbolNames) + io.enumCase(Value, E.Name.str().c_str(), E.Value); +} + +void ScalarBitSetTraits::bitset(IO &io, + CompileSym2Flags &Flags) { + auto FlagNames = getCompileSym2FlagNames(); + for (const auto &E : FlagNames) { + io.bitSetCase(Flags, E.Name.str().c_str(), + static_cast(E.Value)); + } +} + +void ScalarBitSetTraits::bitset(IO &io, + CompileSym3Flags &Flags) { + auto FlagNames = getCompileSym3FlagNames(); + for (const auto &E : FlagNames) { + io.bitSetCase(Flags, E.Name.str().c_str(), + static_cast(E.Value)); + } +} + +void ScalarBitSetTraits::bitset(IO &io, ExportFlags &Flags) { + auto FlagNames = getExportSymFlagNames(); + for (const auto &E : FlagNames) { + io.bitSetCase(Flags, E.Name.str().c_str(), + static_cast(E.Value)); + } +} + +void ScalarBitSetTraits::bitset(IO &io, LocalSymFlags &Flags) { + auto FlagNames = getLocalFlagNames(); + for (const auto &E : FlagNames) { + io.bitSetCase(Flags, E.Name.str().c_str(), + static_cast(E.Value)); + } +} + +void ScalarBitSetTraits::bitset(IO &io, ProcSymFlags &Flags) { + auto FlagNames = getProcSymFlagNames(); + for (const auto &E : FlagNames) { + io.bitSetCase(Flags, E.Name.str().c_str(), + static_cast(E.Value)); + } +} + +void ScalarBitSetTraits::bitset( + IO &io, FrameProcedureOptions &Flags) { + auto FlagNames = getFrameProcSymFlagNames(); + for (const auto &E : FlagNames) { + io.bitSetCase(Flags, E.Name.str().c_str(), + static_cast(E.Value)); + } +} + +void ScalarEnumerationTraits::enumeration(IO &io, CPUType &Cpu) { + auto CpuNames = getCPUTypeNames(); + for (const auto &E : CpuNames) { + io.enumCase(Cpu, E.Name.str().c_str(), static_cast(E.Value)); + } +} + +void ScalarEnumerationTraits::enumeration(IO &io, RegisterId &Reg) { + auto RegNames = getRegisterNames(); + for (const auto &E : RegNames) { + io.enumCase(Reg, E.Name.str().c_str(), static_cast(E.Value)); + } + io.enumFallback(Reg); +} + +void ScalarEnumerationTraits::enumeration( + IO &io, TrampolineType &Tramp) { + auto TrampNames = getTrampolineNames(); + for (const auto &E : TrampNames) { + io.enumCase(Tramp, E.Name.str().c_str(), + static_cast(E.Value)); + } +} + +void ScalarEnumerationTraits::enumeration(IO &io, + ThunkOrdinal &Ord) { + auto ThunkNames = getThunkOrdinalNames(); + for (const auto &E : ThunkNames) { + io.enumCase(Ord, E.Name.str().c_str(), static_cast(E.Value)); + } +} + +namespace llvm { +namespace CodeViewYAML { +namespace detail { + +struct SymbolRecordBase { + codeview::SymbolKind Kind; + explicit SymbolRecordBase(codeview::SymbolKind K) : Kind(K) {} + + virtual ~SymbolRecordBase() {} + virtual void map(yaml::IO &io) = 0; + virtual codeview::CVSymbol + toCodeViewSymbol(BumpPtrAllocator &Allocator) const = 0; + virtual Error fromCodeViewSymbol(codeview::CVSymbol Type) = 0; +}; + +template struct SymbolRecordImpl : public SymbolRecordBase { + explicit SymbolRecordImpl(codeview::SymbolKind K) + : SymbolRecordBase(K), Symbol(static_cast(K)) {} + + void map(yaml::IO &io) override; + + codeview::CVSymbol + toCodeViewSymbol(BumpPtrAllocator &Allocator) const override { + return SymbolSerializer::writeOneSymbol(Symbol, Allocator); + } + Error fromCodeViewSymbol(codeview::CVSymbol CVS) override { + return SymbolDeserializer::deserializeAs(CVS, Symbol); + } + + mutable T Symbol; +}; + +template <> void SymbolRecordImpl::map(IO &IO) {} + +template <> void SymbolRecordImpl::map(IO &IO) { + IO.mapRequired("Parent", Symbol.Parent); + IO.mapRequired("End", Symbol.End); + IO.mapRequired("Next", Symbol.Next); + IO.mapRequired("Off", Symbol.Offset); + IO.mapRequired("Seg", Symbol.Segment); + IO.mapRequired("Len", Symbol.Length); + IO.mapRequired("Ordinal", Symbol.Thunk); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + IO.mapRequired("Type", Symbol.Type); + IO.mapRequired("Size", Symbol.Size); + IO.mapRequired("ThunkOff", Symbol.ThunkOffset); + IO.mapRequired("TargetOff", Symbol.TargetOffset); + IO.mapRequired("ThunkSection", Symbol.ThunkSection); + IO.mapRequired("TargetSection", Symbol.TargetSection); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + IO.mapRequired("SectionNumber", Symbol.SectionNumber); + IO.mapRequired("Alignment", Symbol.Alignment); + IO.mapRequired("Rva", Symbol.Rva); + IO.mapRequired("Length", Symbol.Length); + IO.mapRequired("Characteristics", Symbol.Characteristics); + IO.mapRequired("Name", Symbol.Name); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + IO.mapRequired("Size", Symbol.Size); + IO.mapRequired("Characteristics", Symbol.Characteristics); + IO.mapRequired("Offset", Symbol.Offset); + IO.mapRequired("Segment", Symbol.Segment); + IO.mapRequired("Name", Symbol.Name); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + IO.mapRequired("Ordinal", Symbol.Ordinal); + IO.mapRequired("Flags", Symbol.Flags); + IO.mapRequired("Name", Symbol.Name); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + // TODO: Print the linkage name + + IO.mapRequired("PtrParent", Symbol.Parent); + IO.mapRequired("PtrEnd", Symbol.End); + IO.mapRequired("PtrNext", Symbol.Next); + IO.mapRequired("CodeSize", Symbol.CodeSize); + IO.mapRequired("DbgStart", Symbol.DbgStart); + IO.mapRequired("DbgEnd", Symbol.DbgEnd); + IO.mapRequired("FunctionType", Symbol.FunctionType); + IO.mapRequired("Segment", Symbol.Segment); + IO.mapRequired("Flags", Symbol.Flags); + IO.mapRequired("DisplayName", Symbol.Name); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + IO.mapRequired("Type", Symbol.Index); + IO.mapRequired("Seg", Symbol.Register); + IO.mapRequired("Name", Symbol.Name); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + IO.mapRequired("Type", Symbol.Index); + IO.mapRequired("Seg", Symbol.Segment); + IO.mapRequired("Off", Symbol.Offset); + IO.mapRequired("Name", Symbol.Name); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + IO.mapRequired("SumName", Symbol.SumName); + IO.mapRequired("SymOffset", Symbol.SymOffset); + IO.mapRequired("Mod", Symbol.Module); + IO.mapRequired("Name", Symbol.Name); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + IO.mapRequired("Entries", Symbol.Fields); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + IO.mapRequired("PtrParent", Symbol.Parent); + IO.mapRequired("PtrEnd", Symbol.End); + IO.mapRequired("Inlinee", Symbol.Inlinee); + // TODO: The binary annotations +} + +template <> void SymbolRecordImpl::map(IO &IO) { + IO.mapRequired("Type", Symbol.Type); + IO.mapRequired("Flags", Symbol.Flags); + IO.mapRequired("VarName", Symbol.Name); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + // TODO: Print the subfields +} + +template <> void SymbolRecordImpl::map(IO &IO) { + // TODO: Print the subfields +} + +template <> void SymbolRecordImpl::map(IO &IO) { + // TODO: Print the subfields +} + +template <> void SymbolRecordImpl::map(IO &IO) { + // TODO: Print the subfields +} + +template <> void SymbolRecordImpl::map(IO &IO) { + // TODO: Print the subfields +} + +template <> +void SymbolRecordImpl::map(IO &IO) { + // TODO: Print the subfields +} + +template <> void SymbolRecordImpl::map(IO &IO) { + // TODO: Print the subfields +} + +template <> void SymbolRecordImpl::map(IO &IO) { + // TODO: Print the linkage name + IO.mapRequired("PtrParent", Symbol.Parent); + IO.mapRequired("PtrEnd", Symbol.End); + IO.mapRequired("CodeSize", Symbol.CodeSize); + IO.mapRequired("Segment", Symbol.Segment); + IO.mapRequired("BlockName", Symbol.Name); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + // TODO: Print the linkage name + IO.mapRequired("Segment", Symbol.Segment); + IO.mapRequired("Flags", Symbol.Flags); + IO.mapRequired("Flags", Symbol.Flags); + IO.mapRequired("DisplayName", Symbol.Name); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + IO.mapRequired("Signature", Symbol.Signature); + IO.mapRequired("ObjectName", Symbol.Name); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + IO.mapRequired("Flags", Symbol.Flags); + IO.mapRequired("Machine", Symbol.Machine); + IO.mapRequired("FrontendMajor", Symbol.VersionFrontendMajor); + IO.mapRequired("FrontendMinor", Symbol.VersionFrontendMinor); + IO.mapRequired("FrontendBuild", Symbol.VersionFrontendBuild); + IO.mapRequired("BackendMajor", Symbol.VersionBackendMajor); + IO.mapRequired("BackendMinor", Symbol.VersionBackendMinor); + IO.mapRequired("BackendBuild", Symbol.VersionBackendBuild); + IO.mapRequired("Version", Symbol.Version); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + IO.mapRequired("Flags", Symbol.Flags); + IO.mapRequired("Machine", Symbol.Machine); + IO.mapRequired("FrontendMajor", Symbol.VersionFrontendMajor); + IO.mapRequired("FrontendMinor", Symbol.VersionFrontendMinor); + IO.mapRequired("FrontendBuild", Symbol.VersionFrontendBuild); + IO.mapRequired("FrontendQFE", Symbol.VersionFrontendQFE); + IO.mapRequired("BackendMajor", Symbol.VersionBackendMajor); + IO.mapRequired("BackendMinor", Symbol.VersionBackendMinor); + IO.mapRequired("BackendBuild", Symbol.VersionBackendBuild); + IO.mapRequired("BackendQFE", Symbol.VersionBackendQFE); + IO.mapRequired("Version", Symbol.Version); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + IO.mapRequired("TotalFrameBytes", Symbol.TotalFrameBytes); + IO.mapRequired("PaddingFrameBytes", Symbol.PaddingFrameBytes); + IO.mapRequired("OffsetToPadding", Symbol.OffsetToPadding); + IO.mapRequired("BytesOfCalleeSavedRegisters", + Symbol.BytesOfCalleeSavedRegisters); + IO.mapRequired("OffsetOfExceptionHandler", Symbol.OffsetOfExceptionHandler); + IO.mapRequired("SectionIdOfExceptionHandler", + Symbol.SectionIdOfExceptionHandler); + IO.mapRequired("Flags", Symbol.Flags); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + // TODO: Map Linkage Name + IO.mapRequired("Segment", Symbol.Segment); + IO.mapRequired("Type", Symbol.Type); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + IO.mapRequired("Index", Symbol.Index); + IO.mapRequired("ModFilenameOffset", Symbol.ModFilenameOffset); + IO.mapRequired("Flags", Symbol.Flags); + IO.mapRequired("Name", Symbol.Name); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + // TODO: Map Linkage Name + IO.mapRequired("Segment", Symbol.Segment); + IO.mapRequired("CallInstructionSize", Symbol.CallInstructionSize); + IO.mapRequired("Type", Symbol.Type); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + // TODO: Map Linkage Name + IO.mapRequired("Register", Symbol.Register); + IO.mapRequired("CookieKind", Symbol.CookieKind); + IO.mapRequired("Flags", Symbol.Flags); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + IO.mapRequired("FuncID", Symbol.Indices); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + IO.mapRequired("Type", Symbol.Type); + IO.mapRequired("UDTName", Symbol.Name); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + IO.mapRequired("BuildId", Symbol.BuildId); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + IO.mapRequired("Offset", Symbol.Offset); + IO.mapRequired("Type", Symbol.Type); + IO.mapRequired("VarName", Symbol.Name); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + IO.mapRequired("Offset", Symbol.Offset); + IO.mapRequired("Type", Symbol.Type); + IO.mapRequired("Register", Symbol.Register); + IO.mapRequired("VarName", Symbol.Name); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + IO.mapRequired("Type", Symbol.Type); + IO.mapRequired("Value", Symbol.Value); + IO.mapRequired("Name", Symbol.Name); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + // TODO: Map linkage name + IO.mapRequired("Type", Symbol.Type); + IO.mapRequired("DisplayName", Symbol.Name); +} + +template <> void SymbolRecordImpl::map(IO &IO) { + // TODO: Map linkage name + IO.mapRequired("Type", Symbol.Type); + IO.mapRequired("DisplayName", Symbol.Name); +} +} +} +} + +CVSymbol CodeViewYAML::SymbolRecord::toCodeViewSymbol( + BumpPtrAllocator &Allocator) const { + return Symbol->toCodeViewSymbol(Allocator); +} + +namespace llvm { +namespace yaml { +template <> struct MappingTraits { + static void mapping(IO &io, SymbolRecordBase &Record) { Record.map(io); } +}; +} +} + +template +static inline Expected +fromCodeViewSymbolImpl(CVSymbol Symbol) { + CodeViewYAML::SymbolRecord Result; + + auto Impl = std::make_shared>(Symbol.kind()); + if (auto EC = Impl->fromCodeViewSymbol(Symbol)) + return std::move(EC); + Result.Symbol = Impl; + return Result; +} + +Expected +CodeViewYAML::SymbolRecord::fromCodeViewSymbol(CVSymbol Symbol) { +#define SYMBOL_RECORD(EnumName, EnumVal, ClassName) \ + case EnumName: \ + return fromCodeViewSymbolImpl(Symbol); +#define SYMBOL_RECORD_ALIAS(EnumName, EnumVal, AliasName, ClassName) \ + SYMBOL_RECORD(EnumName, EnumVal, ClassName) + switch (Symbol.kind()) { +#include "llvm/DebugInfo/CodeView/CodeViewSymbols.def" + default: { llvm_unreachable("Unknown symbol kind!"); } + } + return make_error(cv_error_code::corrupt_record); +} + +template +static void mapSymbolRecordImpl(IO &IO, const char *Class, SymbolKind Kind, + CodeViewYAML::SymbolRecord &Obj) { + if (!IO.outputting()) + Obj.Symbol = std::make_shared>(Kind); + + IO.mapRequired(Class, *Obj.Symbol); +} + +void MappingTraits::mapping( + IO &IO, CodeViewYAML::SymbolRecord &Obj) { + SymbolKind Kind; + if (IO.outputting()) + Kind = Obj.Symbol->Kind; + IO.mapRequired("Kind", Kind); + +#define SYMBOL_RECORD(EnumName, EnumVal, ClassName) \ + case EnumName: \ + mapSymbolRecordImpl(IO, #ClassName, Kind, Obj); \ + break; +#define SYMBOL_RECORD_ALIAS(EnumName, EnumVal, AliasName, ClassName) \ + SYMBOL_RECORD(EnumName, EnumVal, ClassName) + switch (Kind) { +#include "llvm/DebugInfo/CodeView/CodeViewSymbols.def" + default: { llvm_unreachable("Unknown symbol kind!"); } + } +} diff --git a/lib/ObjectYAML/CodeViewYAML.cpp b/lib/ObjectYAML/CodeViewYAMLTypes.cpp similarity index 57% rename from lib/ObjectYAML/CodeViewYAML.cpp rename to lib/ObjectYAML/CodeViewYAMLTypes.cpp index 227377785d7..4e82a299a67 100644 --- a/lib/ObjectYAML/CodeViewYAML.cpp +++ b/lib/ObjectYAML/CodeViewYAMLTypes.cpp @@ -1,4 +1,4 @@ -//===- CodeViewYAML.cpp - CodeView YAMLIO implementation ------------------===// +//===- CodeViewYAMLTypes.cpp - CodeView YAMLIO types implementation -------===// // // The LLVM Compiler Infrastructure // @@ -12,12 +12,14 @@ // //===----------------------------------------------------------------------===// -#include "llvm/ObjectYAML/CodeViewYAML.h" +#include "llvm/ObjectYAML/CodeViewYAMLTypes.h" #include "llvm/ADT/StringExtras.h" #include "llvm/ADT/StringSwitch.h" #include "llvm/DebugInfo/CodeView/CVTypeVisitor.h" #include "llvm/DebugInfo/CodeView/CodeViewError.h" #include "llvm/DebugInfo/CodeView/EnumTables.h" +#include "llvm/DebugInfo/CodeView/TypeDeserializer.h" +#include "llvm/DebugInfo/CodeView/TypeTableBuilder.h" using namespace llvm; using namespace llvm::codeview; @@ -25,32 +27,15 @@ using namespace llvm::CodeViewYAML; using namespace llvm::CodeViewYAML::detail; using namespace llvm::yaml; -LLVM_YAML_IS_SEQUENCE_VECTOR(SourceFileChecksumEntry) -LLVM_YAML_IS_SEQUENCE_VECTOR(SourceLineEntry) -LLVM_YAML_IS_SEQUENCE_VECTOR(SourceColumnEntry) -LLVM_YAML_IS_SEQUENCE_VECTOR(SourceLineBlock) -LLVM_YAML_IS_SEQUENCE_VECTOR(SourceLineInfo) -LLVM_YAML_IS_SEQUENCE_VECTOR(InlineeSite) -LLVM_YAML_IS_SEQUENCE_VECTOR(InlineeInfo) LLVM_YAML_IS_SEQUENCE_VECTOR(OneMethodRecord) LLVM_YAML_IS_SEQUENCE_VECTOR(StringRef) LLVM_YAML_IS_SEQUENCE_VECTOR(VFTableSlotKind) LLVM_YAML_IS_FLOW_SEQUENCE_VECTOR(TypeIndex) LLVM_YAML_DECLARE_SCALAR_TRAITS(TypeIndex, false) -LLVM_YAML_DECLARE_SCALAR_TRAITS(CodeViewYAML::HexFormattedString, false) LLVM_YAML_DECLARE_SCALAR_TRAITS(APSInt, false) -LLVM_YAML_DECLARE_MAPPING_TRAITS(CodeViewYAML::SourceLineEntry) -LLVM_YAML_DECLARE_MAPPING_TRAITS(CodeViewYAML::SourceColumnEntry) -LLVM_YAML_DECLARE_MAPPING_TRAITS(CodeViewYAML::SourceFileChecksumEntry) -LLVM_YAML_DECLARE_MAPPING_TRAITS(CodeViewYAML::SourceLineInfo) -LLVM_YAML_DECLARE_MAPPING_TRAITS(CodeViewYAML::SourceLineBlock) -LLVM_YAML_DECLARE_MAPPING_TRAITS(CodeViewYAML::InlineeInfo) -LLVM_YAML_DECLARE_MAPPING_TRAITS(CodeViewYAML::InlineeSite) - LLVM_YAML_DECLARE_ENUM_TRAITS(TypeLeafKind) -LLVM_YAML_DECLARE_ENUM_TRAITS(SymbolKind) LLVM_YAML_DECLARE_ENUM_TRAITS(PointerToMemberRepresentation) LLVM_YAML_DECLARE_ENUM_TRAITS(VFTableSlotKind) LLVM_YAML_DECLARE_ENUM_TRAITS(CallingConvention) @@ -61,28 +46,15 @@ LLVM_YAML_DECLARE_ENUM_TRAITS(MemberAccess) LLVM_YAML_DECLARE_ENUM_TRAITS(MethodKind) LLVM_YAML_DECLARE_ENUM_TRAITS(WindowsRTClassKind) LLVM_YAML_DECLARE_ENUM_TRAITS(LabelType) -LLVM_YAML_DECLARE_ENUM_TRAITS(FileChecksumKind) LLVM_YAML_DECLARE_BITSET_TRAITS(PointerOptions) -LLVM_YAML_DECLARE_BITSET_TRAITS(LineFlags) LLVM_YAML_DECLARE_BITSET_TRAITS(ModifierOptions) LLVM_YAML_DECLARE_BITSET_TRAITS(FunctionOptions) LLVM_YAML_DECLARE_BITSET_TRAITS(ClassOptions) LLVM_YAML_DECLARE_BITSET_TRAITS(MethodOptions) -LLVM_YAML_DECLARE_MAPPING_TRAITS(llvm::codeview::OneMethodRecord) -LLVM_YAML_DECLARE_MAPPING_TRAITS(llvm::codeview::MemberPointerInfo) - -LLVM_YAML_DECLARE_BITSET_TRAITS(CompileSym2Flags) -LLVM_YAML_DECLARE_BITSET_TRAITS(CompileSym3Flags) -LLVM_YAML_DECLARE_BITSET_TRAITS(ExportFlags) -LLVM_YAML_DECLARE_BITSET_TRAITS(LocalSymFlags) -LLVM_YAML_DECLARE_BITSET_TRAITS(ProcSymFlags) -LLVM_YAML_DECLARE_BITSET_TRAITS(FrameProcedureOptions) -LLVM_YAML_DECLARE_ENUM_TRAITS(CPUType) -LLVM_YAML_DECLARE_ENUM_TRAITS(RegisterId) -LLVM_YAML_DECLARE_ENUM_TRAITS(TrampolineType) -LLVM_YAML_DECLARE_ENUM_TRAITS(ThunkOrdinal) +LLVM_YAML_DECLARE_MAPPING_TRAITS(OneMethodRecord) +LLVM_YAML_DECLARE_MAPPING_TRAITS(MemberPointerInfo) namespace llvm { namespace CodeViewYAML { @@ -147,34 +119,6 @@ template struct MemberRecordImpl : public MemberRecordBase { mutable T Record; }; - -struct SymbolRecordBase { - codeview::SymbolKind Kind; - explicit SymbolRecordBase(codeview::SymbolKind K) : Kind(K) {} - - virtual ~SymbolRecordBase() {} - virtual void map(yaml::IO &io) = 0; - virtual codeview::CVSymbol - toCodeViewSymbol(BumpPtrAllocator &Allocator) const = 0; - virtual Error fromCodeViewSymbol(codeview::CVSymbol Type) = 0; -}; - -template struct SymbolRecordImpl : public SymbolRecordBase { - explicit SymbolRecordImpl(codeview::SymbolKind K) - : SymbolRecordBase(K), Symbol(static_cast(K)) {} - - void map(yaml::IO &io) override; - - codeview::CVSymbol - toCodeViewSymbol(BumpPtrAllocator &Allocator) const override { - return SymbolSerializer::writeOneSymbol(Symbol, Allocator); - } - Error fromCodeViewSymbol(codeview::CVSymbol CVS) override { - return SymbolDeserializer::deserializeAs(CVS, Symbol); - } - - mutable T Symbol; -}; } } } @@ -394,175 +338,6 @@ void ScalarBitSetTraits::bitset(IO &IO, MethodOptions &Options) { IO.bitSetCase(Options, "Sealed", MethodOptions::Sealed); } -void ScalarEnumerationTraits::enumeration( - IO &io, FileChecksumKind &Kind) { - io.enumCase(Kind, "None", FileChecksumKind::None); - io.enumCase(Kind, "MD5", FileChecksumKind::MD5); - io.enumCase(Kind, "SHA1", FileChecksumKind::SHA1); - io.enumCase(Kind, "SHA256", FileChecksumKind::SHA256); -} - -void ScalarBitSetTraits::bitset(IO &io, LineFlags &Flags) { - io.bitSetCase(Flags, "HasColumnInfo", LF_HaveColumns); - io.enumFallback(Flags); -} - -void ScalarEnumerationTraits::enumeration(IO &io, - SymbolKind &Value) { - auto SymbolNames = getSymbolTypeNames(); - for (const auto &E : SymbolNames) - io.enumCase(Value, E.Name.str().c_str(), E.Value); -} - -void ScalarBitSetTraits::bitset(IO &io, - CompileSym2Flags &Flags) { - auto FlagNames = getCompileSym2FlagNames(); - for (const auto &E : FlagNames) { - io.bitSetCase(Flags, E.Name.str().c_str(), - static_cast(E.Value)); - } -} - -void ScalarBitSetTraits::bitset(IO &io, - CompileSym3Flags &Flags) { - auto FlagNames = getCompileSym3FlagNames(); - for (const auto &E : FlagNames) { - io.bitSetCase(Flags, E.Name.str().c_str(), - static_cast(E.Value)); - } -} - -void ScalarBitSetTraits::bitset(IO &io, ExportFlags &Flags) { - auto FlagNames = getExportSymFlagNames(); - for (const auto &E : FlagNames) { - io.bitSetCase(Flags, E.Name.str().c_str(), - static_cast(E.Value)); - } -} - -void ScalarBitSetTraits::bitset(IO &io, LocalSymFlags &Flags) { - auto FlagNames = getLocalFlagNames(); - for (const auto &E : FlagNames) { - io.bitSetCase(Flags, E.Name.str().c_str(), - static_cast(E.Value)); - } -} - -void ScalarBitSetTraits::bitset(IO &io, ProcSymFlags &Flags) { - auto FlagNames = getProcSymFlagNames(); - for (const auto &E : FlagNames) { - io.bitSetCase(Flags, E.Name.str().c_str(), - static_cast(E.Value)); - } -} - -void ScalarBitSetTraits::bitset( - IO &io, FrameProcedureOptions &Flags) { - auto FlagNames = getFrameProcSymFlagNames(); - for (const auto &E : FlagNames) { - io.bitSetCase(Flags, E.Name.str().c_str(), - static_cast(E.Value)); - } -} - -void ScalarEnumerationTraits::enumeration(IO &io, CPUType &Cpu) { - auto CpuNames = getCPUTypeNames(); - for (const auto &E : CpuNames) { - io.enumCase(Cpu, E.Name.str().c_str(), static_cast(E.Value)); - } -} - -void ScalarEnumerationTraits::enumeration(IO &io, RegisterId &Reg) { - auto RegNames = getRegisterNames(); - for (const auto &E : RegNames) { - io.enumCase(Reg, E.Name.str().c_str(), static_cast(E.Value)); - } - io.enumFallback(Reg); -} - -void ScalarEnumerationTraits::enumeration( - IO &io, TrampolineType &Tramp) { - auto TrampNames = getTrampolineNames(); - for (const auto &E : TrampNames) { - io.enumCase(Tramp, E.Name.str().c_str(), - static_cast(E.Value)); - } -} - -void ScalarEnumerationTraits::enumeration(IO &io, - ThunkOrdinal &Ord) { - auto ThunkNames = getThunkOrdinalNames(); - for (const auto &E : ThunkNames) { - io.enumCase(Ord, E.Name.str().c_str(), static_cast(E.Value)); - } -} - -void ScalarTraits::output(const HexFormattedString &Value, - void *ctx, raw_ostream &Out) { - StringRef Bytes(reinterpret_cast(Value.Bytes.data()), - Value.Bytes.size()); - Out << toHex(Bytes); -} - -StringRef ScalarTraits::input(StringRef Scalar, void *ctxt, - HexFormattedString &Value) { - std::string H = fromHex(Scalar); - Value.Bytes.assign(H.begin(), H.end()); - return StringRef(); -} - -void MappingTraits::mapping(IO &IO, SourceLineEntry &Obj) { - IO.mapRequired("Offset", Obj.Offset); - IO.mapRequired("LineStart", Obj.LineStart); - IO.mapRequired("IsStatement", Obj.IsStatement); - IO.mapRequired("EndDelta", Obj.EndDelta); -} - -void MappingTraits::mapping(IO &IO, SourceColumnEntry &Obj) { - IO.mapRequired("StartColumn", Obj.StartColumn); - IO.mapRequired("EndColumn", Obj.EndColumn); -} - -void MappingTraits::mapping(IO &IO, SourceLineBlock &Obj) { - IO.mapRequired("FileName", Obj.FileName); - IO.mapRequired("Lines", Obj.Lines); - IO.mapRequired("Columns", Obj.Columns); -} - -void MappingTraits::mapping( - IO &IO, SourceFileChecksumEntry &Obj) { - IO.mapRequired("FileName", Obj.FileName); - IO.mapRequired("Kind", Obj.Kind); - IO.mapRequired("Checksum", Obj.ChecksumBytes); -} - -void MappingTraits::mapping(IO &IO, SourceLineInfo &Obj) { - IO.mapRequired("CodeSize", Obj.CodeSize); - - IO.mapRequired("Flags", Obj.Flags); - IO.mapRequired("RelocOffset", Obj.RelocOffset); - IO.mapRequired("RelocSegment", Obj.RelocSegment); - IO.mapRequired("Blocks", Obj.Blocks); -} - -void MappingTraits::mapping(IO &IO, SourceFileInfo &Obj) { - IO.mapOptional("Checksums", Obj.FileChecksums); - IO.mapOptional("Lines", Obj.LineFragments); - IO.mapOptional("InlineeLines", Obj.Inlinees); -} - -void MappingTraits::mapping(IO &IO, InlineeSite &Obj) { - IO.mapRequired("FileName", Obj.FileName); - IO.mapRequired("LineNum", Obj.SourceLineNum); - IO.mapRequired("Inlinee", Obj.Inlinee); - IO.mapOptional("ExtraFiles", Obj.ExtraFiles); -} - -void MappingTraits::mapping(IO &IO, InlineeInfo &Obj) { - IO.mapRequired("HasExtraFiles", Obj.HasExtraFiles); - IO.mapRequired("Sites", Obj.Sites); -} - void MappingTraits::mapping(IO &IO, MemberPointerInfo &MPI) { IO.mapRequired("ContainingType", MPI.ContainingType); IO.mapRequired("Representation", MPI.Representation); @@ -935,332 +710,3 @@ void MappingTraits::mapping(IO &IO, MemberRecord &Obj) { default: { llvm_unreachable("Unknown member kind!"); } } } - -namespace llvm { -namespace CodeViewYAML { -namespace detail { -template <> void SymbolRecordImpl::map(IO &IO) {} - -template <> void SymbolRecordImpl::map(IO &IO) { - IO.mapRequired("Parent", Symbol.Parent); - IO.mapRequired("End", Symbol.End); - IO.mapRequired("Next", Symbol.Next); - IO.mapRequired("Off", Symbol.Offset); - IO.mapRequired("Seg", Symbol.Segment); - IO.mapRequired("Len", Symbol.Length); - IO.mapRequired("Ordinal", Symbol.Thunk); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - IO.mapRequired("Type", Symbol.Type); - IO.mapRequired("Size", Symbol.Size); - IO.mapRequired("ThunkOff", Symbol.ThunkOffset); - IO.mapRequired("TargetOff", Symbol.TargetOffset); - IO.mapRequired("ThunkSection", Symbol.ThunkSection); - IO.mapRequired("TargetSection", Symbol.TargetSection); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - IO.mapRequired("SectionNumber", Symbol.SectionNumber); - IO.mapRequired("Alignment", Symbol.Alignment); - IO.mapRequired("Rva", Symbol.Rva); - IO.mapRequired("Length", Symbol.Length); - IO.mapRequired("Characteristics", Symbol.Characteristics); - IO.mapRequired("Name", Symbol.Name); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - IO.mapRequired("Size", Symbol.Size); - IO.mapRequired("Characteristics", Symbol.Characteristics); - IO.mapRequired("Offset", Symbol.Offset); - IO.mapRequired("Segment", Symbol.Segment); - IO.mapRequired("Name", Symbol.Name); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - IO.mapRequired("Ordinal", Symbol.Ordinal); - IO.mapRequired("Flags", Symbol.Flags); - IO.mapRequired("Name", Symbol.Name); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - // TODO: Print the linkage name - - IO.mapRequired("PtrParent", Symbol.Parent); - IO.mapRequired("PtrEnd", Symbol.End); - IO.mapRequired("PtrNext", Symbol.Next); - IO.mapRequired("CodeSize", Symbol.CodeSize); - IO.mapRequired("DbgStart", Symbol.DbgStart); - IO.mapRequired("DbgEnd", Symbol.DbgEnd); - IO.mapRequired("FunctionType", Symbol.FunctionType); - IO.mapRequired("Segment", Symbol.Segment); - IO.mapRequired("Flags", Symbol.Flags); - IO.mapRequired("DisplayName", Symbol.Name); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - IO.mapRequired("Type", Symbol.Index); - IO.mapRequired("Seg", Symbol.Register); - IO.mapRequired("Name", Symbol.Name); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - IO.mapRequired("Type", Symbol.Index); - IO.mapRequired("Seg", Symbol.Segment); - IO.mapRequired("Off", Symbol.Offset); - IO.mapRequired("Name", Symbol.Name); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - IO.mapRequired("SumName", Symbol.SumName); - IO.mapRequired("SymOffset", Symbol.SymOffset); - IO.mapRequired("Mod", Symbol.Module); - IO.mapRequired("Name", Symbol.Name); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - IO.mapRequired("Entries", Symbol.Fields); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - IO.mapRequired("PtrParent", Symbol.Parent); - IO.mapRequired("PtrEnd", Symbol.End); - IO.mapRequired("Inlinee", Symbol.Inlinee); - // TODO: The binary annotations -} - -template <> void SymbolRecordImpl::map(IO &IO) { - IO.mapRequired("Type", Symbol.Type); - IO.mapRequired("Flags", Symbol.Flags); - IO.mapRequired("VarName", Symbol.Name); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - // TODO: Print the subfields -} - -template <> void SymbolRecordImpl::map(IO &IO) { - // TODO: Print the subfields -} - -template <> void SymbolRecordImpl::map(IO &IO) { - // TODO: Print the subfields -} - -template <> void SymbolRecordImpl::map(IO &IO) { - // TODO: Print the subfields -} - -template <> void SymbolRecordImpl::map(IO &IO) { - // TODO: Print the subfields -} - -template <> -void SymbolRecordImpl::map(IO &IO) { - // TODO: Print the subfields -} - -template <> void SymbolRecordImpl::map(IO &IO) { - // TODO: Print the subfields -} - -template <> void SymbolRecordImpl::map(IO &IO) { - // TODO: Print the linkage name - IO.mapRequired("PtrParent", Symbol.Parent); - IO.mapRequired("PtrEnd", Symbol.End); - IO.mapRequired("CodeSize", Symbol.CodeSize); - IO.mapRequired("Segment", Symbol.Segment); - IO.mapRequired("BlockName", Symbol.Name); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - // TODO: Print the linkage name - IO.mapRequired("Segment", Symbol.Segment); - IO.mapRequired("Flags", Symbol.Flags); - IO.mapRequired("Flags", Symbol.Flags); - IO.mapRequired("DisplayName", Symbol.Name); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - IO.mapRequired("Signature", Symbol.Signature); - IO.mapRequired("ObjectName", Symbol.Name); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - IO.mapRequired("Flags", Symbol.Flags); - IO.mapRequired("Machine", Symbol.Machine); - IO.mapRequired("FrontendMajor", Symbol.VersionFrontendMajor); - IO.mapRequired("FrontendMinor", Symbol.VersionFrontendMinor); - IO.mapRequired("FrontendBuild", Symbol.VersionFrontendBuild); - IO.mapRequired("BackendMajor", Symbol.VersionBackendMajor); - IO.mapRequired("BackendMinor", Symbol.VersionBackendMinor); - IO.mapRequired("BackendBuild", Symbol.VersionBackendBuild); - IO.mapRequired("Version", Symbol.Version); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - IO.mapRequired("Flags", Symbol.Flags); - IO.mapRequired("Machine", Symbol.Machine); - IO.mapRequired("FrontendMajor", Symbol.VersionFrontendMajor); - IO.mapRequired("FrontendMinor", Symbol.VersionFrontendMinor); - IO.mapRequired("FrontendBuild", Symbol.VersionFrontendBuild); - IO.mapRequired("FrontendQFE", Symbol.VersionFrontendQFE); - IO.mapRequired("BackendMajor", Symbol.VersionBackendMajor); - IO.mapRequired("BackendMinor", Symbol.VersionBackendMinor); - IO.mapRequired("BackendBuild", Symbol.VersionBackendBuild); - IO.mapRequired("BackendQFE", Symbol.VersionBackendQFE); - IO.mapRequired("Version", Symbol.Version); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - IO.mapRequired("TotalFrameBytes", Symbol.TotalFrameBytes); - IO.mapRequired("PaddingFrameBytes", Symbol.PaddingFrameBytes); - IO.mapRequired("OffsetToPadding", Symbol.OffsetToPadding); - IO.mapRequired("BytesOfCalleeSavedRegisters", - Symbol.BytesOfCalleeSavedRegisters); - IO.mapRequired("OffsetOfExceptionHandler", Symbol.OffsetOfExceptionHandler); - IO.mapRequired("SectionIdOfExceptionHandler", - Symbol.SectionIdOfExceptionHandler); - IO.mapRequired("Flags", Symbol.Flags); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - // TODO: Map Linkage Name - IO.mapRequired("Segment", Symbol.Segment); - IO.mapRequired("Type", Symbol.Type); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - IO.mapRequired("Index", Symbol.Index); - IO.mapRequired("ModFilenameOffset", Symbol.ModFilenameOffset); - IO.mapRequired("Flags", Symbol.Flags); - IO.mapRequired("Name", Symbol.Name); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - // TODO: Map Linkage Name - IO.mapRequired("Segment", Symbol.Segment); - IO.mapRequired("CallInstructionSize", Symbol.CallInstructionSize); - IO.mapRequired("Type", Symbol.Type); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - // TODO: Map Linkage Name - IO.mapRequired("Register", Symbol.Register); - IO.mapRequired("CookieKind", Symbol.CookieKind); - IO.mapRequired("Flags", Symbol.Flags); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - IO.mapRequired("FuncID", Symbol.Indices); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - IO.mapRequired("Type", Symbol.Type); - IO.mapRequired("UDTName", Symbol.Name); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - IO.mapRequired("BuildId", Symbol.BuildId); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - IO.mapRequired("Offset", Symbol.Offset); - IO.mapRequired("Type", Symbol.Type); - IO.mapRequired("VarName", Symbol.Name); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - IO.mapRequired("Offset", Symbol.Offset); - IO.mapRequired("Type", Symbol.Type); - IO.mapRequired("Register", Symbol.Register); - IO.mapRequired("VarName", Symbol.Name); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - IO.mapRequired("Type", Symbol.Type); - IO.mapRequired("Value", Symbol.Value); - IO.mapRequired("Name", Symbol.Name); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - // TODO: Map linkage name - IO.mapRequired("Type", Symbol.Type); - IO.mapRequired("DisplayName", Symbol.Name); -} - -template <> void SymbolRecordImpl::map(IO &IO) { - // TODO: Map linkage name - IO.mapRequired("Type", Symbol.Type); - IO.mapRequired("DisplayName", Symbol.Name); -} -} -} -} - -CVSymbol CodeViewYAML::SymbolRecord::toCodeViewSymbol( - BumpPtrAllocator &Allocator) const { - return Symbol->toCodeViewSymbol(Allocator); -} - -namespace llvm { -namespace yaml { -template <> struct MappingTraits { - static void mapping(IO &io, SymbolRecordBase &Record) { Record.map(io); } -}; -} -} - -template -static inline Expected -fromCodeViewSymbolImpl(CVSymbol Symbol) { - CodeViewYAML::SymbolRecord Result; - - auto Impl = std::make_shared>(Symbol.kind()); - if (auto EC = Impl->fromCodeViewSymbol(Symbol)) - return std::move(EC); - Result.Symbol = Impl; - return Result; -} - -Expected -CodeViewYAML::SymbolRecord::fromCodeViewSymbol(CVSymbol Symbol) { -#define SYMBOL_RECORD(EnumName, EnumVal, ClassName) \ - case EnumName: \ - return fromCodeViewSymbolImpl(Symbol); -#define SYMBOL_RECORD_ALIAS(EnumName, EnumVal, AliasName, ClassName) \ - SYMBOL_RECORD(EnumName, EnumVal, ClassName) - switch (Symbol.kind()) { -#include "llvm/DebugInfo/CodeView/CodeViewSymbols.def" - default: { llvm_unreachable("Unknown symbol kind!"); } - } - return make_error(cv_error_code::corrupt_record); -} - -template -static void mapSymbolRecordImpl(IO &IO, const char *Class, SymbolKind Kind, - CodeViewYAML::SymbolRecord &Obj) { - if (!IO.outputting()) - Obj.Symbol = std::make_shared>(Kind); - - IO.mapRequired(Class, *Obj.Symbol); -} - -void MappingTraits::mapping( - IO &IO, CodeViewYAML::SymbolRecord &Obj) { - SymbolKind Kind; - if (IO.outputting()) - Kind = Obj.Symbol->Kind; - IO.mapRequired("Kind", Kind); - -#define SYMBOL_RECORD(EnumName, EnumVal, ClassName) \ - case EnumName: \ - mapSymbolRecordImpl(IO, #ClassName, Kind, Obj); \ - break; -#define SYMBOL_RECORD_ALIAS(EnumName, EnumVal, AliasName, ClassName) \ - SYMBOL_RECORD(EnumName, EnumVal, ClassName) - switch (Kind) { -#include "llvm/DebugInfo/CodeView/CodeViewSymbols.def" - default: { llvm_unreachable("Unknown symbol kind!"); } - } -} diff --git a/tools/llvm-pdbdump/PdbYaml.cpp b/tools/llvm-pdbdump/PdbYaml.cpp index 2cea0f5803d..e288063e2af 100644 --- a/tools/llvm-pdbdump/PdbYaml.cpp +++ b/tools/llvm-pdbdump/PdbYaml.cpp @@ -21,6 +21,7 @@ #include "llvm/DebugInfo/PDB/Native/TpiHashing.h" #include "llvm/DebugInfo/PDB/PDBExtras.h" #include "llvm/DebugInfo/PDB/PDBTypes.h" +#include "llvm/ObjectYAML/CodeViewYAMLTypes.h" using namespace llvm; using namespace llvm::pdb; diff --git a/tools/llvm-pdbdump/PdbYaml.h b/tools/llvm-pdbdump/PdbYaml.h index 2c0b4d8af61..deb500ec207 100644 --- a/tools/llvm-pdbdump/PdbYaml.h +++ b/tools/llvm-pdbdump/PdbYaml.h @@ -19,7 +19,9 @@ #include "llvm/DebugInfo/PDB/Native/PDBFile.h" #include "llvm/DebugInfo/PDB/Native/RawConstants.h" #include "llvm/DebugInfo/PDB/PDBTypes.h" -#include "llvm/ObjectYAML/CodeViewYAML.h" +#include "llvm/ObjectYAML/CodeViewYAMLDebugSections.h" +#include "llvm/ObjectYAML/CodeViewYAMLSymbols.h" +#include "llvm/ObjectYAML/CodeViewYAMLTypes.h" #include "llvm/Support/Endian.h" #include "llvm/Support/YAMLTraits.h" diff --git a/tools/llvm-pdbdump/YAMLOutputStyle.cpp b/tools/llvm-pdbdump/YAMLOutputStyle.cpp index b2e5954c1f9..18839a7679d 100644 --- a/tools/llvm-pdbdump/YAMLOutputStyle.cpp +++ b/tools/llvm-pdbdump/YAMLOutputStyle.cpp @@ -28,7 +28,6 @@ #include "llvm/DebugInfo/PDB/Native/RawConstants.h" #include "llvm/DebugInfo/PDB/Native/RawError.h" #include "llvm/DebugInfo/PDB/Native/TpiStream.h" -#include "llvm/ObjectYAML/CodeViewYAML.h" using namespace llvm; using namespace llvm::codeview; diff --git a/tools/llvm-pdbdump/llvm-pdbdump.cpp b/tools/llvm-pdbdump/llvm-pdbdump.cpp index 0942770c780..0b2b766a3c5 100644 --- a/tools/llvm-pdbdump/llvm-pdbdump.cpp +++ b/tools/llvm-pdbdump/llvm-pdbdump.cpp @@ -61,7 +61,6 @@ #include "llvm/DebugInfo/PDB/PDBSymbolExe.h" #include "llvm/DebugInfo/PDB/PDBSymbolFunc.h" #include "llvm/DebugInfo/PDB/PDBSymbolThunk.h" -#include "llvm/ObjectYAML/CodeViewYAML.h" #include "llvm/Support/BinaryByteStream.h" #include "llvm/Support/COM.h" #include "llvm/Support/CommandLine.h"