<div dir="ltr">Ping. Is this okay?<div>Thanks.<br><div class="gmail_extra"><br><br><div class="gmail_quote">2012/12/18 WenHan Gu (¨¦¨Z¿«) <span dir="ltr"><<a href="mailto:wenhan.gu@mediatek.com" target="_blank">wenhan.gu@mediatek.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I move the logic to caller, which won't push cleanup if not destruct-able.<br>
R4 attached.<br>
<div class="im HOEnZb"><br>
> -----Original Message-----<br>
> From: Eli Friedman [mailto:<a href="mailto:eli.friedman@gmail.com">eli.friedman@gmail.com</a>]<br>
</div><div class="im HOEnZb">> Sent: Tuesday, December 18, 2012 10:15 AM<br>
> To: WenHan Gu (¨¦¨Z¿«)<br>
</div><div class="im HOEnZb">> Cc: John McCall; <a href="mailto:anton@korobeynikov.info">anton@korobeynikov.info</a>; <a href="mailto:andersca@mac.com">andersca@mac.com</a>;<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> Subject: Re: [PATCH] Fix PR14514: Call destructor automatically after a throw<br>
><br>
</div><div class="im HOEnZb">> On Mon, Dec 17, 2012 at 5:53 PM, WenHan Gu (¨¦¨Z¿«)<br>
> <<a href="mailto:wenhan.gu@mediatek.com">wenhan.gu@mediatek.com</a>> wrote:<br>
> > OK, thanks.<br>
> > Patch set 3 uploaded.<br>
><br>
> @@ -1367,6 +1367,9 @@ static void<br>
> emitPartialArrayDestroy(CodeGenFunction &CGF,<br>
>      type = arrayType->getElementType();<br>
>    }<br>
><br>
> +  if (type.isDestructedType() == QualType::DK_none)<br>
> +    return; // Check again for nest array<br>
> +<br>
><br>
> That doesn't help; you're still pushing the cleanup.<br>
><br>
> -Eli<br>
<br>
</div><div class="HOEnZb"><div class="h5">************* Email Confidentiality Notice ********************<br>
The information contained in this e-mail message (including any<br>
attachments) may be confidential, proprietary, privileged, or otherwise<br>
exempt from disclosure under applicable laws. It is intended to be<br>
conveyed only to the designated recipient(s). Any use, dissemination,<br>
distribution, printing, retaining or copying of this e-mail (including its<br>
attachments) by unintended recipient(s) is strictly prohibited and may<br>
be unlawful. If you are not an intended recipient of this e-mail, or believe<br>
that you have received this e-mail in error, please notify the sender<br>
immediately (by replying to this e-mail), delete any and all copies of<br>
this e-mail (including any attachments) from your system, and do not<br>
disclose the content of this e-mail to any other person. Thank you!</div></div><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><br clear="all"><div><br></div>-- <br>Best regards,<div>WenHan Gu (Nowar)</div>
</div></div></div>