On Sun, Dec 16, 2012 at 8:57 PM, WenHan Gu (谷汶翰) <span dir="ltr"><<a href="mailto:wenhan.gu@mediatek.com" target="_blank">wenhan.gu@mediatek.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">






<div lang="ZH-TW" link="blue" vlink="purple">
<div>
<p class="MsoNormal"><span lang="EN-US" style>Hi committers,<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style>This patch fix PR14514:
</span><b><span lang="EN-US" style="background:#d0d0d0;font-size:8.0pt;font-family:"Verdana","sans-serif"">Array of class type didn't call destructor automatically after a throw.<span> </span></span></b><span lang="EN-US" style><u></u><u></u></span></p>

<p class="MsoNormal"><span lang="EN-US" style><a href="http://llvm.org/bugs/show_bug.cgi?id=14514" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=14514</a><u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style>Please have a look.<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style>Thanks!<u></u><u></u></span></p>
</div>
</div>


<table><tbody><tr><td><br></td></tr></tbody></table></blockquote><div><br></div><div>-  assert(!dtor->isTrivial());</div><div>+  if (dtor->isTrivial())</div><div>+    return;</div><div><br></div><div>Something has gone wrong in the other logic in your patch if you need this change.  We shouldn't push cleanups for elements of an array with a trivially destructible type.  You should probably be calling QualType::isDestructedType somewhere.</div>
<div><br></div><div>-Eli</div></div>