[PATCH] D25579: [codeview] emit debug info for indirect virtual base classes

[Rui Ueyama via cfe-commits]

ruiu added inline comments.


================
Comment at: lib/CodeGen/CGDebugInfo.cpp:1392
+    auto *BaseTy = getOrCreateType(BI.getType(), Unit);
+    if (SeenTypes.find(BaseTy) != SeenTypes.end())
+      continue;
----------------
zturner wrote:
> rnk wrote:
> > IMO `SeenTypes.count(...)` would be more idiomatic.
> What a silly function.  I wonder why it isn't called `contains()` and return a `bool`.
It's probably because std::set doesn't provide `contains()` but `count()` too.


https://reviews.llvm.org/D25579