[PATCH] D21410: [pdb] Change type visitor pattern to use dynamic polymorphism

[Reid Kleckner via llvm-commits]

rnk added a comment.

In http://reviews.llvm.org/D21410#460027, @zturner wrote:

> One added advantage of this approach is that, while rebasing, I actually caught a couple of bugs in Rui's patch that would not have been possible to catch at compile time otherwise.  Mainly there were previously separate visitation functions for class, struct, and interface, but these are all implemented by a single record type, and thus have only a single visitation function.  This was caught because when I added the `override` keyword to them, the compiler error'ed saying that there was no suitable function to override.


Acking on the mailing list that I buy this reasoning, lgtm


Repository:
  rL LLVM

http://reviews.llvm.org/D21410