Add concrete type overloads to PDBSymbol::findChildren().

Frequently you only want to iterate over children of a specific
type (e.g. functions).  Previously you would get back a generic
interface that allowed iteration over the base symbol type,
which you would have to dyn_cast<> each one of.  With this patch,
we allow the user to specify the concrete type as a template
parameter, and it will return an iterator which returns instances
of the concrete type directly.

llvm-svn: 228960
diff --git a/llvm/lib/DebugInfo/PDB/PDBSymbolThunk.cpp b/llvm/lib/DebugInfo/PDB/PDBSymbolThunk.cpp
index a7bd96a..eac1a54 100644
--- a/llvm/lib/DebugInfo/PDB/PDBSymbolThunk.cpp
+++ b/llvm/lib/DebugInfo/PDB/PDBSymbolThunk.cpp
@@ -25,11 +25,17 @@
   if (Level == PDB_DumpLevel::Compact) {
     OS.indent(Indent);
     PDB_ThunkOrdinal Ordinal = getThunkOrdinal();
-    OS << "THUNK[" << Ordinal << "] ";
-    OS << "[" << format_hex(getRelativeVirtualAddress(), 10);
+    uint32_t RVA = getRelativeVirtualAddress();
+    if (Ordinal == PDB_ThunkOrdinal::TrampIncremental) {
+      OS << format_hex(RVA, 10);
+    } else {
+      OS << "[" << format_hex(RVA, 10);
+      OS << " - " << format_hex(RVA + getLength(), 10) << "]";
+    }
+    OS << " thunk(" << Ordinal << ")";
     if (Ordinal == PDB_ThunkOrdinal::TrampIncremental)
       OS << " -> " << format_hex(getTargetRelativeVirtualAddress(), 10);
-    OS << "] ";
+    OS << " ";
     std::string Name = getName();
     if (!Name.empty())
       OS << Name;