rjmccall added a comment. Mostly looks fine. One suggestion, though.
================ Comment at: lib/CodeGen/CGVTables.cpp:831 @@ -822,2 +830,3 @@ + return !CGM.getVTables().isVTableExternal(RD); } ---------------- Please do your query second; in practice, many v-tables are not external, meaning that your check will not need to be run. Your check is actually pretty expensive when optimization is enabled because of the full walk scanning for inline functions. http://reviews.llvm.org/D11441 _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
