I've committed a change based on your patch in r186803.<div><br></div><div>A few changes relative to your patch:</div><div> * We preserve the virtual base initializer in a constructor for an abstract class. We'll never use it, but this is important for source fidelity. We also give a warning on this case (disabled by default, since most compilers do not yet implement DR257).</div>
<div> * The handling of implicit deletion of special members was incorrect. The standard has a defect here, which I've reported (there is no special case for virtual bases of abstract classes in the implicit deletion rules), but the exception should only affect constructors. For an assignment operator or a destructor, the virtual base should still be checked.</div>
<div><br></div><div>Thanks!<br><div><br></div><div><div class="gmail_quote">On Fri, Jul 19, 2013 at 3:15 PM, Maurice Bos <span dir="ltr"><<a href="mailto:m-ou.se@m-ou.se" target="_blank">m-ou.se@m-ou.se</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br><br>Attached is a patch to fix <a href="http://llvm.org/bugs/show_bug.cgi?id=16659" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=16659</a>.<br>
<br>Kind regards,<br>-Maurice Bos-<br>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div></div>