Initial pass at API design for DebugInfo/PDB

[zturner]

Summary

*IN PROGRESS* This is in progress and is not supposed to be final. Many interfaces are incomplete / unimplemented. I'm submitting this early to solicit comments on the overall design, so that anyone interested can help shape the design of the library.

I approached this design with the following assumptions in mind:

1. The interface to the PDB reading code should expose no Windows-specific types.
2. LLVMDebugInfoPDB should compile on non-Windows
3. LLVMDebugInfoPDB should support multiple simultaneous implementations of a PDB reader, and you should be able to choose which one you create at runtime.

#1 means that, essentially, I need to "shim" or clone the interface to DIA. A full description of DIA and its available interfaces can be found here: https://msdn.microsoft.com/en-us/library/108e9y6d.aspx. Since DIA is COM-based, there is really no way to hide the Windows nature of these methods and interfaces without making an entirely new interface that looks about the same, and has a DIA version as one possible implementation.

#3 is assumed in order to leave open the possibility that someone in the future may create a different implementation of this PDB reader. Even if that happens though, I believe we may still want a DIA-based version of the itnerface in the code. The reason for this is to make it possible to understand how PDBs evolve with new versions of Visual Studio and new releases of the DIA SDK which expose new functionality which may not be recognized by a non-DIA based implementation.

The DIA SDK has a somewhat awkward object model, so before thinking about this patch it helps to understand at a fundamental level how PDBs are queried in DIA. The full set of interfaces is at the aforementioned URL, but one of those interfaces is of particular importance. The IDiaSymbol interface [https://msdn.microsoft.com/en-us/library/w0edf0x4.aspx]. This is the *only* access to symbols in the PDB. This means functions, executables, global variables, and more are all queried through this interface (A complete list of symbol types is enumerated in the PDB_SymType enum included in this patch). The value of this enum for a particular symbol determines which subset of methods on the IDiaSymbol are valid to call. For example, the methods which are valid to call on an IDiaSymbol whose symbol type is PDB_SymType::Function are documented here [https://msdn.microsoft.com/en-us/library/62w760s9.aspx].

I have attempted to model this in my patch as follows:

1. An abstract interface named IPDBSymbolBase includes the same set of methods defined by IDiaSymbol. Implementers of a particular PDB reading strategy (e.g. DIA) are expected to implement this interface.
2. For each concrete symbol type, a non-abstract class is provided. In this initial patch, this includes PDBFunction and PDBExecutable, which expose the exact set of methods that are valid for that symbol type. This is essentially a wrapper about the base type, whose purpose is to limit the set of methods which the user can call.
3. A user can obtain one of the concrete wrapper types by either manually constructing it with an IPDBSymbolBase pointer, or by using a dyn_cast<> like template called pdb_symbol_cast<>.

Note that DIA provides access to other information from a PDB that is not just symbols. For example, line numbers, source files, and various other things are simply independent interfaces and do not follow this tag-based / casting model. For those, I have simply cloned the interface and will expect the implementor to provide an implementation, and the user to use it through a base pointer.

Sorry for the long winded introduction, but since some people on this list have 0 experience with DIA, I hope this helps. I'm open to completely gutting this / re-working it if people think a different design would be more appropriate.

[zturner]

The value of this enum for a particular symbol determines which subset of methods on the IDiaSymbol are valid to call.

One additional point about this line. The subset of methods which are valid to call for a particular IDiaSymbol is not easy to organize into a useful object hierarchy. For some values of the tag field, the subsets will be disjoint, while for others they will overlap, and for others they might even be the same. If you take the intersection across all tag values, for example, the only common method will be the function which actually returns the tag value.

So it's difficult to organize this into a meaningful object hierarchy.

[dblaikie]

Few random comments provided.

include/llvm/DebugInfo/PDB/IPDBDataStream.h

24

In theory the LLVM coding convention requires that types with vtables have an anchor function (one non-inline virtual function).

I'm not sure if the type /only/ has pure virtual functions whether that matters (but it probably does - even a vtable of only pure virtual functions still has to exist, I think)

26

This is the total number of things this stream will visit? (or the number remaining? or the current index?)

27

What's this the name of?

28

I take it this returns the current item but doesn't move the cursor? Maybe it should be const?

29

return Optional<RecordType> rather than an out parameter?

(I'd probably call this "next" but that might be a bit vague (& collides with std::next))

include/llvm/DebugInfo/PDB/IPDBSourceFile.h

31

missing newline at end of file

include/llvm/DebugInfo/PDB/IPDBSymbolBase.h

26

Wouldn't this return value be passing ownership (via unique_ptr or similar)? and similarly on the next two functions.

include/llvm/DebugInfo/PDB/PDBTypes.h

28

Not sure if these typedefs help readability - it feels like they hinder it by hiding a common construct (unique_ptr) in something opaque when reading the code. But I tend to be a bit verbose...

[zturner]

include/llvm/DebugInfo/PDB/IPDBDataStream.h

26

The former. It's the total number of times it's valid to call getNext() if you're at the beginning of the stream (when it's newly constructed, or you've just called reset).

27

The name of the data stream (which in turn identifies what kind of data it has). The stream named "FPO" for example, contains fpo data.

28

In theory it probably could be, but the underlying DIA interface doesn't have any const methods. I could probably use const on almost every single method of IPDBSymbol if we're ok const_casting. I didn't make anything const though since the underlying DIA interface has no const methods.

Since you poitned out this method though, I also noticed that I left off the index.

include/llvm/DebugInfo/PDB/IPDBSymbolBase.h

26

Yes, thanks. I forgot to update these to return unique_ptrs.

include/llvm/DebugInfo/PDB/PDBTypes.h

28

Does llvm tend to write out unique_ptr<T> everywhere? It leads to a lot of line wrapping.

[dblaikie]

include/llvm/DebugInfo/PDB/IPDBDataStream.h

26

Not sure if there might be a better name for this. (size()?)

27

Is that useful to clients? Would there be a better API like just having different types (templated, perhaps) of streams?

28

Chances are you won't need to const_cast much since you'll have indirection to the underlying API anyway, I assume (the concrete implementation of this interface will have a member pointer to some DIA structure, right? And you call functions on the thing pointed to - since const isn't "deep" that'll just work without const_cast)

(open question about the threading guarantees of the underlying DIA APIs, though - if those operations that are conceptually const but not marked const in the DIA API are not thread safe, that might surprise users of this API to call a non-thread-safe const function, but if that's the case, a comment that this whole API is thread incompatible would be fine I think)

include/llvm/DebugInfo/PDB/PDBTypes.h

28

I think so, at least in my experience - but I don't claim to have lots of data.

(I tend to be much more conservative about using typedefs (rarely use them) than most people, so feel free to treat this feedback with a grain of salt)

[zturner]

include/llvm/DebugInfo/PDB/IPDBDataStream.h

27

I actually pondered not including access to the underlying streams at all. The reason is that the streams simply provide access to the raw bytes in the PDB file. All of the information present in the streams is available through the typed methods as well. But I figured it was useful still for the dumper. From what I can tell, the dumper will be the only real client of the stream data, so just returning the name sems kind of reasonable. Another thing is that it's not documented what all of the different streams that might be returned are. So far I've observed a few different ones through some dumps I've run on my own, but they're not always the same, so there could be some we don't know about.

[zturner]

As an aside, one thing I actually thought about with regards to the object model was only including 1 method in the base class. the one that gets the symbol tag. Then having each concrete type derive from the base tag and then provide pure virtual methods for the methods that it would support. So instead of having this PDBFunction, for example, which is ultimately just a facade that forwards methods to the underlying monolithic interface, instead have an IPDBFunction whose vtable contains the exact set of methods relevant for a function.

The advantages of this approach is that

1. you can no longer call invalid methods through the IPDBSymbolBase interface.
2. pdb_symbol_cast<> has more natural behavior now. Instead of casting a pointer type to a value type, it actually behaves more like a traditional dynamic cast.

And the disadvantages are:

1. Implementor has to do more work now. It has to implement the same function many times, for example when valid methods overlap.

[zturner]

Addressed all (I think?) of David's comments.

[dblaikie]

include/llvm/DebugInfo/PDB/PDBCasting.h

28

This cast machinery seems a bit confusing owing to the mix of pointers and value types. (& you mention in the summary that you can also use the traditional llvm::*cast machinery too? That would be odd as well - to cast to concrete types from pointer types)

One other way that PDBFunction et, al could be implemented that might be more natural (& require less of this machinery) - make PDBSymbol opaque (virtual dtor, no other functions) and have various PDBFunction/etc derive from that. Use LLVM's standard cast machinery here. Implement them in terms of a PDBRawSymbol/PDBSymbolImpl, something like that (which would be what your current PDBSymbol is) - maybe the base opaque PDBSymbol could have a non-virtual "getRaw/getImpl/whatever" to access the raw thing when desired (in dumpers, etc).

I just hesitate to add more 'odd' casting machinery - we already stretch that idiom a bit further than I think is helpful with some of our hierarchies.

lib/DebugInfo/PDB/PDBInterfaceAnchors.cpp

21

I don't think you need an out-of-line ctor /and/ dtor, just the dtor would be fine - and maybe a comment about these dtors being anchors.

[zturner]

Reworked the class hierarchy per David's suggestions. Library now should work with llvm::isa<> and llvm::dyn_cast<> instead of relying on custom casting machinery. This was done by injecting a new base class below all the concrete PDBFunction / etc types.

[dblaikie]

include/llvm/DebugInfo/PDB/IPDBEnumChildren.h

25

Name's OK, but "count" still seems like it might be a bit ambiguous with index, or number remaining. Another option (which seems more idiomatic in LLVM) might be "getNumChildren" (while I would like "size()" it's annoying that the count is attached to the iterator and not to some separate structure that produces iterators and I don't know how to properly communicate the separation of these concepts, unfortunately)

include/llvm/DebugInfo/PDB/IPDBRawSymbol.h

193 ↗

(On Diff #19275)

How're these functions going to communicate failure?

include/llvm/DebugInfo/PDB/PDBSymbol.h

33 ↗

(On Diff #19275)

return by reference rather than pointer here, perhaps?

46 ↗

(On Diff #19275)

could probably make this private and the derived classes can just call "getRawSymbol" - the derived classes don't need the ability to mutate (null, repoint, etc) this variable. (optional, not a big deal either way, just a little more stringent encapsulation)

The member could actually be const, too/instead.

lib/DebugInfo/PDB/PDB.cpp

21

"return nullptr" will work here

[zturner]

include/llvm/DebugInfo/PDB/IPDBRawSymbol.h

193 ↗

(On Diff #19275)

Undefined in theory, probably false in practice. You shouldn't call methods on the raw interface unless you know it's a valid method.

For the purposes of a dumper who wanted to detect unknown / unexpected fields, that knowledge could all be encapsulated in the implementation of the raw interface. For example, you could have PDBSymbol::dump() which calls RawSymbol->dump(), and that particular implementation can go to the native API instead of calling the friendly accessors.

[zturner]

I was thinking it might still be useful for people who knew what they were
doing and who wanted to write some implementation-specific code to be able
to get at the native native interface. Like in this case a raw COM
pointer. In theory this shouldn't ever be necessary, but it's hard to say
without knowing anything about implementation strategies other than DIA how
much compatibility someone would be able to achieve, and if there ever
might be things that are only possible to implement in terms of one API but
not the other. So I didn't want to prematurely remove the possibility to
get at the native interface, for example by writing
static_cast<DIASymbol*>(PDBSymbol::getRawSymbol())->someDIAOnlyMethod().

[zturner]

Ok this CL just got way bigger, but it should be fairly complete in terms of stuff that will live in the generic API. There is now one concrete implementation of PDBSymbol for every tag value, and this uses some macro C++11 magic to forward the calls through to the underlying raw interface.

Additionally, this provides a unit test which uses a mock implementation of the underlying raw symbol to verify that the dyncast semantics are as expected. Confirmed that the unit tests are hooked up correctly, as I changed one line in the test to something that would obviously fail, and my test failed.

[zturner]

ping. Don't expect you to read through this whole patch again thoroughly since it's gotten long, but I think all issues have been addressed. And many of the files are fairly boilerplate. LMK if you're ok with this going in.

[dblaikie]

Few optional bits of feedback but otherwise seems fine.

If you want some more help/details on the .def stuff, let me know - happy to go through it.

lib/DebugInfo/PDB/PDBInterfaceAnchors.cpp

65

This still means the vtable is emitted weakly, though it will only be emitted in one place. So this doesn't quite meet the style guide's requirement & I'd rather use the virtual dtor or an explicit anchor function, but I'm not so fussed. I do think the rule is a bit silly, but I'm not sure this particular approach is worthwhile (I suppose my thought is that the rule is silly enough that it's not worth partly/tangentially/sort-of adhering to, might as well just not bother adhering to it at all, perhaps - I dunno)

lib/DebugInfo/PDB/PDBSymbol.cpp

56 ↗

(On Diff #19361)

Could possibly move the #define closer to the use (even inside the function) and maybe undef it afterwards.

63 ↗

(On Diff #19361)

Would it be useful to do this more as a .def file style - Take a look at Duncan's recent use of include/llvm/IR/Metadata.def for an example (essentially have the big long list of all the PDBSymbols written once, in one file - then #define a macro for what you want to do with those names, then include the file - thus stamping out all sorts of things you want, for all PDBSymbols)

unittests/DebugInfo/PDB/PDBApiTest.cpp

53 ↗

(On Diff #19361)

Probably prefer the LHS return type. The only reason to use the -> return type formulation is if you want to refer to function parameters, generally.

Also, rather than casting nullptr, the usual way to "give me a thing of some particular kind in an unevaluated context" is a utility called declval:

decltype(declval<IPDBRawSymbol>().Func()) Func() const override {

62 ↗

(On Diff #19361)

Use override rather than virtual (for all of these).

include/llvm/DebugInfo/PDB/IPDBDataStream.h

+//===- IPDBDataStream.h - base interface for child enumerator -*- C++ -*-===//
+//
+//                     The LLVM Compiler Infrastructure
+//
+// This file is distributed under the University of Illinois Open Source
+// License. See LICENSE.TXT for details.
+//
+//===----------------------------------------------------------------------===//
+
+#ifndef LLVM_DEBUGINFO_PDB_IPDBDATASTREAM_H
+#define LLVM_DEBUGINFO_PDB_IPDBDATASTREAM_H
+
+#include "llvm/ADT/SmallVector.h"
+#include "llvm/ADT/Optional.h"
+
+#include "PDBTypes.h"
+
+namespace llvm {
+
+class IPDBDataStream {
+public:
+  typedef llvm::SmallVector<uint8_t, 32> RecordType;
+
+  IPDBDataStream();
+  virtual ~IPDBDataStream();
+
+  virtual uint32_t getRecordCount() const = 0;
+  virtual std::string getName() const = 0;
+  virtual llvm::Optional<RecordType> getItemAtIndex(uint32_t Index) const = 0;
+  virtual bool getNext(RecordType &Record) = 0;
+  virtual void reset() = 0;
+  virtual PDBDataStreamPtr clone() const = 0;
+};
+}
+
+#endif

include/llvm/DebugInfo/PDB/IPDBEnumChildren.h

+//===- IPDBEnumChildren.h - base interface for child enumerator -*- C++ -*-===//
+//
+//                     The LLVM Compiler Infrastructure
+//
+// This file is distributed under the University of Illinois Open Source
+// License. See LICENSE.TXT for details.
+//
+//===----------------------------------------------------------------------===//
+
+#ifndef LLVM_DEBUGINFO_PDB_IPDBENUMCHILDREN_H
+#define LLVM_DEBUGINFO_PDB_IPDBENUMCHILDREN_H
+
+#include "PDBTypes.h"
+
+namespace llvm {
+
+template <class ChildType> class IPDBEnumChildren {
+public:
+  typedef std::unique_ptr<ChildType> ChildTypePtr;
+  typedef std::unique_ptr<IPDBEnumChildren<Childtype>> MyTypePtr;
+
+  virtual ~IPDBEnumChildren() {}
+
+  virtual uint32 getChildCount() const = 0;
+  virtual ChildTypePtr getChildAtIndex(uint32_t Index) const = 0;
+  virtual ChildTypePtr getNext() = 0;
+  virtual void reset() = 0;
+  virtual MyTypePtr clone() const = 0;
+};
+}
+
+#endif

include/llvm/DebugInfo/PDB/IPDBSession.h

+//===- IPDBSession.h - base interface for a PDB symbol context --*- C++ -*-===//
+//
+//                     The LLVM Compiler Infrastructure
+//
+// This file is distributed under the University of Illinois Open Source
+// License. See LICENSE.TXT for details.
+//
+//===----------------------------------------------------------------------===//
+
+#ifndef LLVM_DEBUGINFO_PDB_IPDBSESSION_H
+#define LLVM_DEBUGINFO_PDB_IPDBSESSION_H
+
+#include "PDBTypes.h"
+#include "PDBExecutable.h"
+
+namespace llvm {
+
+class IPDBSession {
+public:
+  IPDBSession();
+  virtual ~IPDBSession();
+
+  virtual uint64_t getLoadAddress() const = 0;
+  virtual void setLoadAddress(uint64_t Address) = 0;
+  virtual PDBExecutable getGlobalScope() const = 0;
+  virtual PDBSymbolPtr getSymbolById() const = 0;
+  virtual PDBSourceFilePtr getSourceFileById() const = 0;
+
+  virtual PDBEnumDataStreamsPtr getDebugStreams() const = 0;
+};
+}
+
+#endif
 No newline at end of file

include/llvm/DebugInfo/PDB/IPDBSourceFile.h

+//===- IPDBSourceFile.h - base interface for a PDB source file --*- C++ -*-===//
+//
+//                     The LLVM Compiler Infrastructure
+//
+// This file is distributed under the University of Illinois Open Source
+// License. See LICENSE.TXT for details.
+//
+//===----------------------------------------------------------------------===//
+
+#ifndef LLVM_DEBUGINFO_PDB_IPDBSOURCEFILE_H
+#define LLVM_DEBUGINFO_PDB_IPDBSOURCEFILE_H
+
+#include "PDBTypes.h"
+
+namespace llvm {
+
+class IPDBSourceFile {
+public:
+  IPDBSourceFile();
+  virtual ~IPDBSourceFile();
+
+  virtual std::string getFileName() const = 0;
+  virtual uint32_t getUniqueId() const = 0;
+  virtual std::string getChecksum() const = 0;
+  virtual PDB_Checksum getChecksumType() const = 0;
+  virtual PDBEnumSymbolsPtr getCompilands() const = 0;
+};
+}
+
+#endif
 No newline at end of file

include/llvm/DebugInfo/PDB/IPDBSymbolBase.h

+//===- IPDBSymbolBase.h - ase interface for PDB symbol types ----*- C++ -*-===//
+//
+//                     The LLVM Compiler Infrastructure
+//
+// This file is distributed under the University of Illinois Open Source
+// License. See LICENSE.TXT for details.
+//
+//===----------------------------------------------------------------------===//
+
+#ifndef LLVM_DEBUGINFO_PDB_IPDBSYMBOLBASE_H
+#define LLVM_DEBUGINFO_PDB_IPDBSYMBOLBASE_H
+
+#include "llvm/ADT/StringRef.h"
+#include "llvm/Support/COFF.h"
+#include "PDBTypes.h"
+
+namespace llvm {
+
+class IPDBSymbolEnumerator;
+
+class IPDBSymbolBase {
+public:
+  IPDBSymbolBase();
+  virtual ~IPDBSymbolBase();
+
+  virtual PDBEnumSymbolsPtr findChildren(PDB_SymType Type, StringRef Name,
+                                         PDB_NameSearchFlags Flags) const = 0;
+  virtual PDBEnumSymbolsPtr findChildrenByRVA(PDB_SymType Type, StringRef Name,
+                                              PDB_NameSearchFlags Flags,
+                                              uint32_t RVA) const = 0;
+  virtual PDBEnumSymbolsPtr findInlineFramesByRVA(uint32_t RVA) const = 0;
+
+  virtual PDB_MemberAccess getAccess() const = 0;
+  virtual uint32_t getAddressOffset() const = 0;
+  virtual uint32_t getAddressSection() const = 0;
+  virtual uint32_t getAge() const = 0;
+  virtual uint32_t getArrayIndexTypeId() const = 0;
+  virtual uint32_t getBackEndBuild() const = 0;
+  virtual uint32_t getBackEndMajor() const = 0;
+  virtual uint32_t getBackEndMinor() const = 0;
+  virtual uint32_t getBaseDataOffset() const = 0;
+  virtual uint32_t getBaseDataSlot() const = 0;
+  virtual uint32_t getBaseSymbolId() const = 0;
+  virtual PDB_BuiltinType getBuiltinType() const = 0;
+  virtual uint32_t getBitPosition() const = 0;
+  virtual PDB_CallingConv getCallingConvention() const = 0;
+  virtual uint32_t getClassParentId() const = 0;
+  virtual std::string getCompilerName() const = 0;
+  virtual uint32_t getCount() const = 0;
+  virtual uint32_t getCountLiveRanges() const = 0;
+  virtual uint32_t getFrontEndBuild() const = 0;
+  virtual uint32_t getFrontEndMajor() const = 0;
+  virtual uint32_t getFrontEndMinor() const = 0;
+  virtual PDB_Lang getLanguage() const = 0;
+  virtual uint32_t getLexicalParentId() const = 0;
+  virtual std::string getLibraryName() const = 0;
+  virtual uint32_t getLiveRangeStartAddressOffset() const = 0;
+  virtual uint32_t getLiveRangeStartAddressSection() const = 0;
+  virtual uint32_t getLiveRangeStartRelativeVirtualAddress() const = 0;
+  virtual uint32_t getLocalBasePointerRegisterId() const = 0;
+  virtual uint32_t getLowerBoundId() const = 0;
+  virtual uint32_t getMemorySpaceKind() const = 0;
+  virtual std::string getName() const = 0;
+  virtual uint32_t getNumberOfAcceleratorPointerTags() const = 0;
+  virtual uint32_t getNumberOfColumns() const = 0;
+  virtual uint32_t getNumberOfModifiers() const = 0;
+  virtual uint32_t getNumberOfRegisterIndices() const = 0;
+  virtual uint32_t getNumberOfRows() const = 0;
+  virtual std::string getObjectFileName() const = 0;
+  virtual uint32_t getOemSymbolId() const = 0;
+  virtual uint32_t getOffsetInUdt() const = 0;
+  virtual PDB_Cpu getPlatform() const = 0;
+  virtual uint32_t getRank() const = 0;
+  virtual uint32_t getRegisterId() const = 0;
+  virtual uint32_t getRegisterType() const = 0;
+  virtual uint32_t getRelativeVirtualAddress() const = 0;
+  virtual uint32_t getSamplerSlot() const = 0;
+  virtual uint32_t getSignature() const = 0;
+  virtual uint32_t getSizeInUdt() const = 0;
+  virtual uint32_t getSlot() const = 0;
+  virtual std::string getSourceFileName() const = 0;
+  virtual uint32_t getStride() const = 0;
+  virtual uint32_t getSubTypeId() const = 0;
+  virtual std::string getSymbolsFileName() const = 0;
+  virtual uint32_t getSymIndexId() const = 0;
+  virtual uint32_t getTargetOffset() const = 0;
+  virtual uint32_t getTargetRelativeVirtualAddress() const = 0;
+  virtual uint32_t getTargetSection() const = 0;
+  virtual uint32_t getTextureSlot() const = 0;
+  virtual uint32_t getTimeStamp() const = 0;
+  virtual uint32_t getToken() const = 0;
+  virtual uint32_t getTypeId() const = 0;
+  virtual uint32_t getUavSlot() const = 0;
+  virtual std::string getUndecoratedName() const = 0;
+  virtual uint32_t getUnmodifiedTypeId() const = 0;
+  virtual uint32_t getUpperBoundId() const = 0;
+  virtual uint32_t getVirtualBaseDispIndex() const = 0;
+  virtual uint32_t getVirtualBaseOffset() const = 0;
+  virtual uint32_t getVirtualTableShapeId() const = 0;
+  virtual PDB_DataKind getDataKind() const = 0;
+  virtual PDB_SymType getSymTag() const = 0;
+  virtual PDB_UniqueId getGuid() const = 0;
+  virtual int32_t getOffset() const = 0;
+  virtual int32_t getThisAdjust() const = 0;
+  virtual int32_t getVirtualBasePointerOffset() const = 0;
+  virtual PDB_LocType getLocationType() const = 0;
+  virtual COFF::MachineTypes getMachineType() const = 0;
+  virtual PDB_ThunkOrdinal getThunkOrdinal() const = 0;
+  virtual uint64_t getLength() const = 0;
+  virtual uint64_t getLiveRangeLength() const = 0;
+  virtual uint64_t getTargetVirtualAddress() const = 0;
+  virtual uint64_t getVirtualAddress() const = 0;
+  virtual PDB_UdtType getUdtKind() const = 0;
+  virtual bool hasConstructor() const = 0;
+  virtual bool hasCustomCallingConvention() const = 0;
+  virtual bool hasFarReturn() const = 0;
+  virtual bool isCode() const = 0;
+  virtual bool isCompilerGenerated() const = 0;
+  virtual bool isConstType() const = 0;
+  virtual bool isEditAndContinueEnabled() const = 0;
+  virtual bool isFunction() const = 0;
+  virtual bool getAddressTaken() const = 0;
+  virtual bool getNoStackOrdering() const = 0;
+  virtual bool hasAlloca() const = 0;
+  virtual bool hasAssignmentOperator() const = 0;
+  virtual bool hasCTypes() const = 0;
+  virtual bool hasCastOperator() const = 0;
+  virtual bool hasDebugInfo() const = 0;
+  virtual bool hasEH() const = 0;
+  virtual bool hasEHa() const = 0;
+  virtual bool hasInlAsm() const = 0;
+  virtual bool hasInlineAttribute() const = 0;
+  virtual bool hasInterruptReturn() const = 0;
+  virtual bool hasLongJump() const = 0;
+  virtual bool hasManagedCode() const = 0;
+  virtual bool hasNestedTypes() const = 0;
+  virtual bool hasNoInlineAttribute() const = 0;
+  virtual bool hasNoReturnAttribute() const = 0;
+  virtual bool hasOptimizedCodeDebugInfo() const = 0;
+  virtual bool hasOverloadedOperator() const = 0;
+  virtual bool hasSEH() const = 0;
+  virtual bool hasSecurityChecks() const = 0;
+  virtual bool hasSetJump() const = 0;
+  virtual bool hasStrictGSCheck() const = 0;
+  virtual bool isAcceleratorGroupSharedLocal() const = 0;
+  virtual bool isAcceleratorPointerTagLiveRange() const = 0;
+  virtual bool isAcceleratorStubFunction() const = 0;
+  virtual bool isAggregated() const = 0;
+  virtual bool isIntroVirtualFunction() const = 0;
+  virtual bool isCVTCIL() const = 0;
+  virtual bool isConstructorVirtualBase() const = 0;
+  virtual bool isCxxReturnUdt() const = 0;
+  virtual bool isDataAligned() const = 0;
+  virtual bool isHLSLData() const = 0;
+  virtual bool isHotpatchable() const = 0;
+  virtual bool isIndirectVirtualBaseClass() const = 0;
+  virtual bool isInterfaceUdt() const = 0;
+  virtual bool isIntrinsic() const = 0;
+  virtual bool isLTCG() const = 0;
+  virtual bool isLocationControlFlowDependent() const = 0;
+  virtual bool isMSILNetmodule() const = 0;
+  virtual bool isManagedRef() const = 0;
+  virtual bool isMatrixRowMajor() const = 0;
+  virtual bool isMsilRef() const = 0;
+  virtual bool isMultipleInheritance() const = 0;
+  virtual bool isNaked() const = 0;
+  virtual bool isNested() const = 0;
+  virtual bool isOptimizedAway() const = 0;
+  virtual bool isPacked() const = 0;
+  virtual bool isPointerBasedOnSymbolValue() const = 0;
+  virtual bool isPointerToDataMember() const = 0;
+  virtual bool isPointerToMemberFunction() const = 0;
+  virtual bool isPureVirtual() const = 0;
+  virtual bool isRValueReference() const = 0;
+  virtual bool isRefUdt() const = 0;
+  virtual bool isReference() const = 0;
+  virtual bool isRestrictedType() const = 0;
+  virtual bool isReturnValue() const = 0;
+  virtual bool isSafeBuffers() const = 0;
+  virtual bool isScoped() const = 0;
+  virtual bool isSdl() const = 0;
+  virtual bool isSingleInheritance() const = 0;
+  virtual bool isSplitted() const = 0;
+  virtual bool isStatic() const = 0;
+  virtual bool hasPrivateSymbols() const = 0;
+  virtual bool isUnalignedType() const = 0;
+  virtual bool isUnreached() const = 0;
+  virtual bool isValueUdt() const = 0;
+  virtual bool isVirtual() const = 0;
+  virtual bool isVirtualBaseClass() const = 0;
+  virtual bool isVirtualInheritance() const = 0;
+  virtual bool isVolatileType() const = 0;
+};
+
+} // namespace llvm
+
+#endif

include/llvm/DebugInfo/PDB/PDB.h

+//===- PDB.h - base header file for creating a PDB reader -------*- C++ -*-===//
+//
+//                     The LLVM Compiler Infrastructure
+//
+// This file is distributed under the University of Illinois Open Source
+// License. See LICENSE.TXT for details.
+//
+//===----------------------------------------------------------------------===//
+
+#ifndef LLVM_DEBUGINFO_PDB_PDB_H
+#define LLVM_DEBUGINFO_PDB_PDB_H
+
+#include "PDBTypes.h"
+
+namespace llvm {
+class StringRef;
+
+PDBSessionPtr createPDBReader(PDB_ReaderType Type, StringRef Path);
+}
+
+#endif

include/llvm/DebugInfo/PDB/PDBCasting.h

+//===- PDBCasting.h - functions for casting PDB symbols ---------*- C++ -*-===//
+//
+//                     The LLVM Compiler Infrastructure
+//
+// This file is distributed under the University of Illinois Open Source
+// License. See LICENSE.TXT for details.
+//
+//===----------------------------------------------------------------------===//
+
+#ifndef LLVM_DEBUGINFO_PDB_PDBCASTING_H
+#define LLVM_DEBUGINFO_PDB_PDBCASTING_H
+
+#include <cassert>
+#include <utility>
+
+#include "PDBExecutable.h"
+#include "PDBFunction.h"
+#include "PDBTypes.h"
+
+namespace llvm {
+
+template <typename T> bool pdb_symbol_isa(PDBSymbolPtr Symbol) { return false; }
+
+template <> bool pdb_symbol_isa<PDBFunction>(PDBSymbolPtr Symbol) {
+  return Symbol->getSymTag() == PDB_SymType::Function;
+}
+
+template <> bool pdb_symbol_isa<PDBExecutable>(PDBSymbolPtr Symbol) {
+  return Symbol->getSymTag() == PDB_SymType::Executable;
+}
+
+/// Similar to dyn_cast<>, this converts a PDBSymbolPtr to the correct "derived"
+/// type.  The derived type is a wrapper around the PDBSymbolPtr and assumes
+/// ownership.
+template <typename T> T pdb_symbol_cast(PDBSymbolPtr Symbol) {
+  assert(Symbol && pdb_symbol_isa<T>(Symbol) &&
+         "pdb_symbol_cast<> invalid conversion");
+  return T(std::move(Symbol));
+}
+
+template <typename T> T pdb_symbol_cast(IPDBSymbolBase *Symbol) {
+  assert(Symbol && pdb_symbol_isa<T>(Symbol) &&
+         "pdb_symbol_cast<> invalid conversion");
+  return T(PDBSymbolPtr(Symbol));
+}
+}
+
+#endif

include/llvm/DebugInfo/PDB/PDBExecutable.h

+//===- PDBExecutable.h - Accessors for querying executables in a PDB ----*-===//
+//
+//                     The LLVM Compiler Infrastructure
+//
+// This file is distributed under the University of Illinois Open Source
+// License. See LICENSE.TXT for details.
+//
+//===----------------------------------------------------------------------===//
+#ifndef LLVM_DEBUGINFO_PDB_PDBEXECUTABLE_H
+#define LLVM_DEBUGINFO_PDB_PDBEXECUTABLE_H
+
+#include <string>
+
+#include "llvm/Support/COFF.h"
+
+#include "PDBTypes.h"
+
+namespace llvm {
+
+class IPDBSymbolBase;
+
+class PDBExecutable {
+public:
+  PDBExecutable(PDBSymbolPtr ExeSymbol);
+
+  uint32_t getAge();
+  PDB_UniqueId getGuid();
+  bool hasCTypes();
+  bool hasPrivateSymbols();
+  COFF::MachineTypes getMachineType();
+  std::string getName();
+  uint32_t getSignature();
+  std::string getSymbolsFileName();
+  uint32_t getSymIndexId();
+
+private:
+  PDBSymbolPtr Executable;
+};
+}
+
+#endif

include/llvm/DebugInfo/PDB/PDBFunction.h

+//===- PDBFunction.h - class representing a function instance ---*- C++ -*-===//
+//
+//                     The LLVM Compiler Infrastructure
+//
+// This file is distributed under the University of Illinois Open Source
+// License. See LICENSE.TXT for details.
+//
+//===----------------------------------------------------------------------===//
+
+#ifndef LLVM_DEBUGINFO_PDB_PDBFUNCTION_H
+#define LLVM_DEBUGINFO_PDB_PDBFUNCTION_H
+
+#include "IPDBSymbolBase.h"
+
+namespace llvm {
+
+class IPDBSymbolBase;
+
+class PDBFunction {
+public:
+  PDBFunction(PDBSymbolPtr FuncSymbol);
+
+  PDB_MemberAccess getAccess();
+  uint32_t getAddressOffset();
+  uint32_t getAddressSection();
+  uint32_t getClassParentId();
+  bool isConstType();
+  bool hasCustomCallingConvention();
+  bool hasFarReturn();
+  bool hasAlloca();
+  bool hasEH();
+  bool hasEHa();
+  bool hasInlAsm();
+  bool hasLongJump();
+  bool hasSEH();
+  bool hasSecurityChecks();
+  bool hasSetJump();
+  bool hasInterruptReturn();
+  bool isIntroVirtualFunction();
+  bool hasInlineAttribute();
+  bool isNaked();
+  bool isStatic();
+  uint64_t getLength();
+  uint32_t getLexicalParentId();
+  PDB_LocType getLocationType();
+  std::string getName();
+  bool hasNoInlineAttribute();
+  bool hasNoReturnAttribute();
+  bool isUnreached();
+  bool getNoStackOrdering();
+  bool hasOptimizedCodeDebugInfo();
+  bool isPureVirtual();
+  uint32_t getRelativeVirtualAddress();
+  uint32_t getSymIndexId();
+  uint32_t getToken();
+  uint32_t getTypeId();
+  bool isUnalignedType();
+  std::string getUndecoratedName();
+  bool isVirtual();
+  uint64_t getVirtualAddress();
+  uint32_t getVirtualBaseOffset();
+  bool isVolatileType();
+
+private:
+  PDBSymbolPtr Function;
+};
+
+}; // namespace llvm
+
+#endif // LLVM_DEBUGINFO_PDB_PDBFUNCTION_H

include/llvm/DebugInfo/PDB/PDBTypes.h

+//===- PDBTypes.h - Defines enums for various fields contained in PDB ---*-===//
+//
+//                     The LLVM Compiler Infrastructure
+//
+// This file is distributed under the University of Illinois Open Source
+// License. See LICENSE.TXT for details.
+//
+//===----------------------------------------------------------------------===//
+
+#ifndef LLVM_DEBUGINFO_PDB_PDBTYPES_H
+#define LLVM_DEBUGINFO_PDB_PDBTYPES_H
+
+#include <memory>
+
+namespace llvm {
+
+class IPDBDataStream;
+template <class T> class IPDBEnumChildren;
+class IPDBSession;
+class IPDBSourceFile;
+class IPDBSymbolBase;
+
+typedef IPDBEnumChildren<IPDBSymbolBase> IPDBEnumSymbols;
+typedef IPDBEnumChildren<IPDBSourceFile> IPDBEnumSourceFiles;
+typedef IPDBEnumChildren<IPDBDataStream> IPDBEnumDataStreams;
+
+typedef std::unique_ptr<IPDBDataStream> PDBDataStreamPtr;
+typedef std::unique_ptr<IPDBSession> PDBSessionPtr;
+typedef std::unique_ptr<IPDBSourceFile> PDBSourceFilePtr;
+typedef std::unique_ptr<IPDBSymbolBase> PDBSymbolPtr;
+
+typedef std::unique_ptr<IPDBEnumSymbols> PDBEnumSymbolsPtr;
+typedef std::unique_ptr<IPDBEnumSourceFiles> PDBEnumSourceFilesPtr;
+typedef std::unique_ptr<IPDBEnumDataStreams> PDBEnumDataStreamsPtr;
+
+class PDBFunction;
+class PDBExecutable;
+
+/// Specifies which PDB reader implementation is to be used.  Only a value
+/// of PDB_ReaderType::DIA is supported.
+enum class PDB_ReaderType {
+  SystemDefault = 0,
+#if defined(_MSC_VER)
+  DIA = 1,
+#endif
+};
+
+/// Defines a 128-bit unique identifier.  This maps to a GUID on Windows, but
+/// is abstracted here for the purposes of non-Windows platforms that don't have
+/// the GUID structure defined.
+struct PDB_UniqueId {
+  uint64_t HighPart;
+  uint64_t LowPart;
+};
+
+/// An enumeration indicating the type of data contained in this table.
+enum class PDB_TableType {
+  Symbols,
+  SourceFiles,
+  LineNumbers,
+  SectionContribs,
+  Segments,
+  InjectedSources,
+  FrameData
+};
+
+/// Defines flags used for enumerating child symbols.  This corresponds to the
+/// NameSearchOptions enumeration which is documented here:
+/// https://msdn.microsoft.com/en-us/library/yat28ads.aspx
+enum PDB_NameSearchFlags {
+  NS_Default = 0x0,
+  NS_CaseSensitive = 0x1,
+  NS_CaseInsensitive = 0x2,
+  NS_FileNameExtMatch = 0x4,
+  NS_Regex = 0x8,
+  NS_UndecoratedName = 0x10
+};
+
+/// Specifies the hash algorithm that a source file from a PDB was hashed with.
+/// This corresponds to the CV_SourceChksum_t enumeration and are documented
+/// here: https://msdn.microsoft.com/en-us/library/e96az21x.aspx
+enum class PDB_Checksum { None = 0, MD5 = 1, SHA1 = 2 };
+
+/// These values correspond to the CV_CPU_TYPE_e enumeration, and are documented
+/// here: https://msdn.microsoft.com/en-us/library/b2fc64ek.aspx
+enum class PDB_Cpu {
+  Intel8080 = 0x0,
+  Intel8086 = 0x1,
+  Intel80286 = 0x2,
+  Intel80386 = 0x3,
+  Intel80486 = 0x4,
+  Pentium = 0x5,
+  PentiumPro = 0x6,
+  Pentium3 = 0x7,
+  MIPS = 0x10,
+  MIPS16 = 0x11,
+  MIPS32 = 0x12,
+  MIPS64 = 0x13,
+  MIPSI = 0x14,
+  MIPSII = 0x15,
+  MIPSIII = 0x16,
+  MIPSIV = 0x17,
+  MIPSV = 0x18,
+  M68000 = 0x20,
+  M68010 = 0x21,
+  M68020 = 0x22,
+  M68030 = 0x23,
+  M68040 = 0x24,
+  Alpha = 0x30,
+  Alpha21164 = 0x31,
+  Alpha21164A = 0x32,
+  Alpha21264 = 0x33,
+  Alpha21364 = 0x34,
+  PPC601 = 0x40,
+  PPC603 = 0x41,
+  PPC604 = 0x42,
+  PPC620 = 0x43,
+  PPCFP = 0x44,
+  PPCBE = 0x45,
+  SH3 = 0x50,
+  SH3E = 0x51,
+  SH3DSP = 0x52,
+  SH4 = 0x53,
+  SHMedia = 0x54,
+  ARM3 = 0x60,
+  ARM4 = 0x61,
+  ARM4T = 0x62,
+  ARM5 = 0x63,
+  ARM5T = 0x64,
+  ARM6 = 0x65,
+  ARM_XMAC = 0x66,
+  ARM_WMMX = 0x67,
+  ARM7 = 0x68,
+  Omni = 0x70,
+  Ia64 = 0x80,
+  Ia64_2 = 0x81,
+  CEE = 0x90,
+  AM33 = 0xa0,
+  M32R = 0xb0,
+  TriCore = 0xc0,
+  X64 = 0xd0,
+  EBC = 0xe0,
+  Thumb = 0xf0,
+  ARMNT = 0xf4,
+  D3D11_Shader = 0x100,
+};
+
+/// These values correspond to the CV_call_e enumeration, and are documented
+/// at the following locations:
+///   https://msdn.microsoft.com/en-us/library/b2fc64ek.aspx
+///   https://msdn.microsoft.com/en-us/library/windows/desktop/ms680207(v=vs.85).aspx
+///
+enum class PDB_CallingConv {
+  NearCdecl = 0x00,
+  FarCdecl = 0x01,
+  NearPascal = 0x02,
+  FarPascal = 0x03,
+  NearFastcall = 0x04,
+  FarFastcall = 0x05,
+  Skipped = 0x06,
+  NearStdcall = 0x07,
+  FarStdcall = 0x08,
+  NearSyscall = 0x09,
+  FarSyscall = 0x0a,
+  Thiscall = 0x0b,
+  MipsCall = 0x0c,
+  Generic = 0x0d,
+  Alphacall = 0x0e,
+  Ppccall = 0x0f,
+  SuperHCall = 0x10,
+  Armcall = 0x11,
+  AM33call = 0x12,
+  Tricall = 0x13,
+  Sh5call = 0x14,
+  M32R = 0x15,
+  Clrcall = 0x16,
+  Inline = 0x17,
+  NearVectorcall = 0x18,
+  Reserved = 0x19,
+};
+
+/// These values correspond to the CV_CFL_LANG enumeration, and are documented
+/// here: https://msdn.microsoft.com/en-us/library/bw3aekw6.aspx
+enum class PDB_Lang {
+  C = 0x00,
+  Cpp = 0x01,
+  Fortran = 0x02,
+  Masm = 0x03,
+  Pascal = 0x04,
+  Basic = 0x05,
+  Cobol = 0x06,
+  Link = 0x07,
+  Cvtres = 0x08,
+  Cvtpgd = 0x09,
+  CSharp = 0x0a,
+  VB = 0x0b,
+  ILAsm = 0x0c,
+  Java = 0x0d,
+  JScript = 0x0e,
+  MSIL = 0x0f,
+  HLSL = 0x10
+};
+
+/// These values correspond to the DataKind enumeration, and are documented
+/// here: https://msdn.microsoft.com/en-us/library/b2x2t313.aspx
+enum class PDB_DataKind {
+  Unknown,
+  Local,
+  StaticLocal,
+  Param,
+  ObjectPtr,
+  FileStatic,
+  Global,
+  Member,
+  StaticMember,
+  Constant
+};
+
+/// These values correspond to the SymTagEnum enumeration, and are documented
+/// here: https://msdn.microsoft.com/en-us/library/bkedss5f.aspx
+enum class PDB_SymType {
+  None,
+  Exe,
+  Compiland,
+  CompilandDetails,
+  CompilandEnv,
+  Function,
+  Block,
+  Data,
+  Annotation,
+  Label,
+  PublicSymbol,
+  UDT,
+  Enum,
+  FunctionSig,
+  PointerType,
+  ArrayType,
+  BuiltinType,
+  Typedef,
+  BaseClass,
+  Friend,
+  FunctionArg,
+  FuncDebugStart,
+  FuncDebugEnd,
+  UsingNamespace,
+  VTableShape,
+  VTable,
+  Custom,
+  Thunk,
+  CustomType,
+  ManagedType,
+  Dimension,
+  CallSite,
+  InlineSite,
+  BaseInterface,
+  VectorType,
+  MatrixType,
+  HLSLType,
+  Caller,
+  Callee,
+  Max
+};
+
+/// These values correspond to the LocationType enumeration, and are documented
+/// here: https://msdn.microsoft.com/en-us/library/f57kaez3.aspx
+enum class PDB_LocType {
+  Null,
+  Static,
+  TLS,
+  RegRel,
+  ThisRel,
+  Enregistered,
+  BitField,
+  Slot,
+  IlRel,
+  MetaData,
+  nstant,
+  Max
+};
+
+/// These values correspond to the THUNK_ORDINAL enumeration, and are documented
+/// here: https://msdn.microsoft.com/en-us/library/dh0k8hft.aspx
+enum class PDB_ThunkOrdinal {
+  Standard,
+  ThisAdjustor,
+  Vcall,
+  Pcode,
+  UnknownLoad,
+  TrampIncremental,
+  BranchIsland
+};
+
+/// These values correspond to the UdtKind enumeration, and are documented
+/// here: https://msdn.microsoft.com/en-us/library/wcstk66t.aspx
+enum class PDB_UdtType { Struct, Class, Union, Interface };
+
+/// These values correspond to the StackFrameTypeEnum enumeration, and are
+/// documented here: https://msdn.microsoft.com/en-us/library/bc5207xw.aspx.
+enum class PDB_StackFrameType { FPO, KernelTrap, KernelTSS, EBP, FrameData };
+
+/// These values correspond to the StackFrameTypeEnum enumeration, and are
+/// documented here: https://msdn.microsoft.com/en-us/library/bc5207xw.aspx.
+enum class PDB_MemoryType { Code, Data, Stack, HeapCode };
+
+/// These values correspond to the Basictype enumeration, and are documented
+/// here: https://msdn.microsoft.com/en-us/library/4szdtzc3.aspx
+enum class PDB_BuiltinType {
+  None = 0,
+  Void = 1,
+  Char = 2,
+  WCharT = 3,
+  Int = 6,
+  UInt = 7,
+  Float = 8,
+  BCD = 9,
+  Bool = 10,
+  Long = 13,
+  ULong = 14,
+  Currency = 25,
+  Date = 26,
+  Variant = 27,
+  Complex = 28,
+  Bitfield = 29,
+  BSTR = 30,
+  HResult = 31
+};
+
+enum class PDB_MemberAccess { Private = 1, Protected = 2, Public = 3 };
+
+} // namespace llvm
+
+#endif

lib/DebugInfo/CMakeLists.txt

 
-add_subdirectory(DWARF)
+add_subdirectory(DWARF)
+if (MSVC)
+  add_subdirectory(PDB)
+endif()
 No newline at end of file

lib/DebugInfo/LLVMBuild.txt

 ;===------------------------------------------------------------------------===;
 
 [common]
-subdirectories = DWARF
+subdirectories = DWARF PDB
 
 [component_0]
 type = Group

lib/DebugInfo/PDB/CMakeLists.txt

+
+if(MSVC AND NOT(MSVC_VERSION LESS 1800))
+  set(MSVC_DIA_SDK_DIR "$ENV{VSINSTALLDIR}DIA SDK")
+
+  # Due to a bug in MSVC 2013's installation software, it is possible
+  # for MSVC 2013 to write the DIA SDK into the Visual Studio 2012
+  # install directory.  If this happens, the installation is corrupt
+  # and there's nothing we can do.  It happens with enough frequency
+  # though that we should handle it.  We do so by simply checking that
+  # the DIA SDK folder exists.  Should this happen you will need to
+  # uninstall VS 2012 and then re-install VS 2013.
+  if (IS_DIRECTORY ${MSVC_DIA_SDK_DIR})
+    include_directories(${MSVC_DIA_SDK_DIR}/include)
+    if (CMAKE_SIZEOF_VOID_P EQUAL 8)
+      link_directories(${MSVC_DIA_SDK_DIR}/lib/amd64)
+    else()
+      link_directories(${MSVC_DIA_SDK_DIR}/lib)
+    endif()
+    set(LIBPDB_ADDITIONAL_LIBRARIES "diaguids")
+  endif()
+endif()
+
+add_llvm_library(LLVMDebugInfoPDB
+  PDB.cpp
+  PDBFunction.cpp
+  PDBInterfaceAnchors.cpp
+  PDBExecutable.cpp
+  )
+  
+target_link_libraries(LLVMDebugInfoPDB INTERFACE "${LIBPDB_ADDITIONAL_LIBRARIES}")
+set(LLVM_LINK_COMPONENTS
+  Support
+  )

lib/DebugInfo/PDB/LLVMBuild.txt

+;===- ./lib/DebugInfo/PDB/LLVMBuild.txt ------------------------*- Conf -*--===;
+;
+;                     The LLVM Compiler Infrastructure
+;
+; This file is distributed under the University of Illinois Open Source
+; License. See LICENSE.TXT for details.
+;
+;===------------------------------------------------------------------------===;
+;
+; This is an LLVMBuild description file for the components in this subdirectory.
+;
+; For more information on the LLVMBuild system, please see:
+;
+;   http://llvm.org/docs/LLVMBuild.html
+;
+;===------------------------------------------------------------------------===;
+
+[component_0]
+type = Library
+name = DebugInfoPDB
+parent = DebugInfo
+required_libraries = Support
+

lib/DebugInfo/PDB/PDB.cpp

+//===- PDB.cpp - base header file for creating a PDB reader -----*- C++ -*-===//
+//
+//                     The LLVM Compiler Infrastructure
+//
+// This file is distributed under the University of Illinois Open Source
+// License. See LICENSE.TXT for details.
+//
+//===----------------------------------------------------------------------===//
+
+#include "llvm/ADT/StringRef.h"
+
+#include "llvm/DebugInfo/PDB/PDB.h"
+#include "llvm/DebugInfo/PDB/IPDBSession.h"
+
+using namespace llvm;
+
+PDBSessionPtr llvm::createPDBReader(PDB_ReaderType Type, StringRef Path) {
+  // Create the correct concrete instance type based on the value of Type.
+  return PDBSessionPtr();
+}

lib/DebugInfo/PDB/PDBExecutable.cpp

+//===- PDBExecutable.cpp - Accessors for querying executables in a PDB --*-===//
+//
+//                     The LLVM Compiler Infrastructure
+//
+// This file is distributed under the University of Illinois Open Source
+// License. See LICENSE.TXT for details.
+//
+//===----------------------------------------------------------------------===//
+
+#include <utility>
+
+#include "llvm/DebugInfo/PDB/IPDBSymbolBase.h"
+#include "llvm/DebugInfo/PDB/PDBExecutable.h"
+
+using namespace llvm;
+
+PDBExecutable::PDBExecutable(PDBSymbolPtr ExeSymbol)
+    : Executable(std::move(ExeSymbol)) {}
+
+uint32_t PDBExecutable::getAge() { return Executable->getAge(); }
+
+PDB_UniqueId PDBExecutable::getGuid() { return Executable->getGuid(); }
+
+bool PDBExecutable::hasCTypes() { return Executable->hasCTypes(); }
+
+bool PDBExecutable::hasPrivateSymbols() {
+  return Executable->hasPrivateSymbols();
+}
+
+COFF::MachineTypes PDBExecutable::getMachineType() {
+  return Executable->getMachineType();
+}
+
+std::string PDBExecutable::getName() { return Executable->getName(); }
+
+uint32_t PDBExecutable::getSignature() { return Executable->getSignature(); }
+
+std::string PDBExecutable::getSymbolsFileName() {
+  return Executable->getSymbolsFileName();
+}
+
+uint32_t PDBExecutable::getSymIndexId() { return Executable->getSymIndexId(); }

lib/DebugInfo/PDB/PDBFunction.cpp

+//===- PDBFunction.cpp - Accessors for querying functions in a PDB ------*-===//
+//
+//                     The LLVM Compiler Infrastructure
+//
+// This file is distributed under the University of Illinois Open Source
+// License. See LICENSE.TXT for details.
+//
+//===----------------------------------------------------------------------===//
+
+#include <utility>
+
+#include "llvm/DebugInfo/PDB/PDBFunction.h"
+#include "llvm/DebugInfo/PDB/IPDBSymbolBase.h"
+
+using namespace llvm;
+
+PDBFunction::PDBFunction(PDBSymbolPtr FuncSymbol)
+    : Function(std::move(FuncSymbol)) {}
+
+PDB_MemberAccess PDBFunction::getAccess() { return Function->getAccess(); }
+
+uint32_t PDBFunction::getAddressOffset() {
+  return Function->getAddressOffset();
+}
+
+uint32_t PDBFunction::getAddressSection() {
+  return Function->getAddressSection();
+}
+
+uint32_t PDBFunction::getClassParentId() {
+  return Function->getClassParentId();
+}
+
+bool PDBFunction::isConstType() { return Function->isConstType(); }
+
+bool PDBFunction::hasCustomCallingConvention() {
+  return Function->hasCustomCallingConvention();
+}
+
+bool PDBFunction::hasFarReturn() { return Function->hasFarReturn(); }
+
+bool PDBFunction::hasAlloca() { return Function->hasAlloca(); }
+
+bool PDBFunction::hasEH() { return Function->hasEH(); }
+
+bool PDBFunction::hasEHa() { return Function->hasEHa(); }
+
+bool PDBFunction::hasInlAsm() { return Function->hasInlAsm(); }
+
+bool PDBFunction::hasLongJump() { return Function->hasLongJump(); }
+
+bool PDBFunction::hasSEH() { return Function->hasSEH(); }
+
+bool PDBFunction::hasSecurityChecks() { return Function->hasSecurityChecks(); }
+
+bool PDBFunction::hasSetJump() { return Function->hasSetJump(); }
+
+bool PDBFunction::hasInterruptReturn() {
+  return Function->hasInterruptReturn();
+}
+
+bool PDBFunction::isIntroVirtualFunction() {
+  return Function->isIntroVirtualFunction();
+}
+
+bool PDBFunction::hasInlineAttribute() {
+  return Function->hasInlineAttribute();
+}
+
+bool PDBFunction::isNaked() { return Function->isNaked(); }
+
+bool PDBFunction::isStatic() { return Function->isStatic(); }
+
+uint64_t PDBFunction::getLength() { return Function->getLength(); }
+
+uint32_t PDBFunction::getLexicalParentId() {
+  return Function->getLexicalParentId();
+}
+
+PDB_LocType PDBFunction::getLocationType() {
+  return Function->getLocationType();
+}
+
+std::string PDBFunction::getName() { return Function->getName(); }
+
+bool PDBFunction::hasNoInlineAttribute() {
+  return Function->hasNoInlineAttribute();
+}
+
+bool PDBFunction::hasNoReturnAttribute() {
+  return Function->hasNoReturnAttribute();
+}
+
+bool PDBFunction::isUnreached() { return Function->isUnreached(); }
+
+bool PDBFunction::getNoStackOrdering() {
+  return Function->getNoStackOrdering();
+}
+
+bool PDBFunction::hasOptimizedCodeDebugInfo() {
+  return Function->hasOptimizedCodeDebugInfo();
+}
+
+bool PDBFunction::isPureVirtual() { return Function->isPureVirtual(); }
+
+uint32_t PDBFunction::getRelativeVirtualAddress() {
+  return Function->getRelativeVirtualAddress();
+}
+
+uint32_t PDBFunction::getSymIndexId() { return Function->getSymIndexId(); }
+
+uint32_t PDBFunction::getToken() { return Function->getToken(); }
+
+uint32_t PDBFunction::getTypeId() { return Function->getTypeId(); }
+
+bool PDBFunction::isUnalignedType() { return Function->isUnalignedType(); }
+
+std::string PDBFunction::getUndecoratedName() {
+  return Function->getUndecoratedName();
+}
+
+bool PDBFunction::isVirtual() { return Function->isVirtual(); }
+
+uint64_t PDBFunction::getVirtualAddress() {
+  return Function->getVirtualAddress();
+}
+
+uint32_t PDBFunction::getVirtualBaseOffset() {
+  return Function->getVirtualBaseOffset();
+}
+
+bool PDBFunction::isVolatileType() { return Function->isVolatileType(); }

lib/DebugInfo/PDB/PDBInterfaceAnchors.cpp

+//===- PDBInterfaceAnchors.h - defines class anchor funcions ----*- C++ -*-===//
+//
+//                     The LLVM Compiler Infrastructure
+//
+// This file is distributed under the University of Illinois Open Source
+// License. See LICENSE.TXT for details.
+//
+//===----------------------------------------------------------------------===//
+// Class anchors are necessary per the LLVM Coding style guide, to ensure that
+// the vtable is only generated in this object file, and not in every object
+// file that incldues the corresponding header.
+//===----------------------------------------------------------------------===//
+
+#include "llvm/DebugInfo/PDB/IPDBDataStream.h"
+#include "llvm/DebugInfo/PDB/IPDBSession.h"
+#include "llvm/DebugInfo/PDB/IPDBSourceFile.h"
+#include "llvm/DebugInfo/PDB/IPDBSymbolBase.h"
+
+using namespace llvm;
+
+IPDBSession::IPDBSession() {}
+
+IPDBSession::~IPDBSession() {}
+
+IPDBDataStream::IPDBDataStream() {}
+
+IPDBDataStream::~IPDBDataStream() {}
+
+IPDBSymbolBase::IPDBSymbolBase() {}
+
+IPDBSymbolBase::~IPDBSymbolBase() {}
+
+IPDBSourceFile::IPDBSourceFile() {}
+
+IPDBSourceFile::~IPDBSourceFile() {}