Prazek marked 3 inline comments as done.

================
Comment at: lib/CodeGen/CGVTables.cpp:835
@@ -834,6 +834,3 @@
 
-  typedef std::vector<const CXXRecordDecl *>::const_iterator const_iterator;
-  for (const_iterator i = DeferredVTables.begin(),
-                      e = DeferredVTables.end(); i != e; ++i) {
-    const CXXRecordDecl *RD = *i;
+  for (const auto &RD : DeferredVTables) {
     if (shouldEmitVTableAtEndOfTranslationUnit(*this, RD))
----------------
majnemer wrote:
> It would be nicer to use `const CXXRecordDecl *RD : DeferredVTables` because 
> the type in the collection isn't trivially obvious.  Also, the braces for the 
> `for-loop` are now superfluous.
const CXXRecordDecl *const RD.



http://reviews.llvm.org/D11446




_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to