[PATCH] D21410: [pdb] Change type visitor pattern to use dynamic polymorphism
Zachary Turner via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 16 10:47:35 PDT 2016
zturner updated this revision to Diff 60994.
zturner added a comment.
Rebases on top of Rui's recent patches.
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.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 55258 bytes
Desc: not available
More information about the llvm-commits