[llvm-pdbdump] Minor prepatory refactor of Class Def Dumper.

In a followup patch I intend to introduce an additional dumping
mode which dumps a graphical representation of a class's layout.
In preparation for this, the text-based layout printer needs to
be split out from the graphical layout printer, and both need
to be able to use the same code for printing the intro and outro
of a class's definition (e.g. base class list, etc).

This patch does so, and in the process introduces a skeleton
definition for the graphical printer, while currently making
the graphical printer just print nothing.

NFC

llvm-svn: 300134

1 files changed, 9 insertions, 4 deletions

diff --git a/llvm/lib/DebugInfo/PDB/UDTLayout.cpp b/llvm/lib/DebugInfo/PDB/UDTLayout.cpp
index 71443fe7587..525913c1865 100644
--- a/llvm/lib/DebugInfo/PDB/UDTLayout.cpp
+++ b/llvm/lib/DebugInfo/PDB/UDTLayout.cpp
@@ -89,7 +89,8 @@ VTableLayoutItem::VTableLayoutItem(const UDTLayoutBase &Parent,
           VTableType->getPointeeType())) {
     VTableFuncs.resize(Shape->getCount());
 
-    auto ParentFunctions = Parent.getSymbol().findAllChildren<PDBSymbolFunc>();
+    auto ParentFunctions =
+        Parent.getSymbolBase().findAllChildren<PDBSymbolFunc>();
     while (auto Func = ParentFunctions->getNext()) {
       if (Func->isVirtual()) {
         uint32_t Index = Func->getVirtualBaseOffset();
@@ -109,15 +110,19 @@ VTableLayoutItem::VTableLayoutItem(const UDTLayoutBase &Parent,
 
 UDTLayoutBase::UDTLayoutBase(const PDBSymbol &Symbol, const std::string &Name,
                              uint32_t Size)
-    : Symbol(Symbol), Name(Name), SizeOf(Size) {
+    : SymbolBase(Symbol), Name(Name), SizeOf(Size) {
   UsedBytes.resize(Size);
   ChildrenPerByte.resize(Size);
   initializeChildren(Symbol);
 }
 
+ClassLayout::ClassLayout(const PDBSymbolTypeUDT &UDT)
+    : UDTLayoutBase(UDT, UDT.getName(), UDT.getLength()), UDT(UDT) {}
+
 ClassLayout::ClassLayout(std::unique_ptr<PDBSymbolTypeUDT> UDT)
-    : UDTLayoutBase(*UDT, UDT->getName(), UDT->getLength()),
-      Type(std::move(UDT)) {}
+    : ClassLayout(*UDT) {
+  OwnedStorage = std::move(UDT);
+}
 
 BaseClassLayout::BaseClassLayout(const UDTLayoutBase &Parent,
                                  std::unique_ptr<PDBSymbolTypeBaseClass> Base)