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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits