<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Sat, Aug 29, 2015 at 12:23 PM Olivier Goffart via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
Please review the attached patch.<br>
<br>
In Sema::BuildCXXFunctionalCastExpr, if the class has a destructor, the<br>
Op.SrcExpr might be a CXXBindTemporaryExpr which we need to unwrap.<br>
<br>
In the testcase, the first new CHECK worked (because A does not have a<br>
destructor),  but the second CHECK failed (did not include the last<br>
parenthese) because D has a destructor.<br>
<br>
I used dyn_cast_or_null just to be safe, becasue i don't know if it is<br>
possible for the BindExpr->getSubExpr() to be null.<br></blockquote><div><br></div><div>Do you know why the added test for A says 'class foo::A' instead of 'foo::A' as the other tests do in that file?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Thanks<br>
--<br>
Olivier<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</blockquote></div></div>