Looks good if andersca or someone familiar w/ this bit of codegen agrees...
================
Comment at: lib/CodeGen/CGExprCXX.cpp:1739-1740
@@ +1738,4 @@
+
+ // Accumulate the base class offsets.
+ if (NumPublicPaths == 0) {
+ const ASTRecordLayout &Layout = Context.getASTRecordLayout(J->Class);
----------------
continue instead?
I'm almost tempted to suggest unswitching this, and having two for loops...
http://llvm-reviews.chandlerc.com/D364
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits