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

Reply via email to