
<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">On Jul 31, 2013, at 7:41 AM, Timur Iskhodzhanov <<a href="mailto:timurrrr@google.com">timurrrr@google.com</a>> wrote:<br><div><blockquote type="cite"><div style="font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">This small patch fixes<span class="Apple-converted-space"> </span><a href="http://llvm.org/PR16735">http://llvm.org/PR16735</a><br><br>I've decided to go the same way Reid went with EmitCXXDestructors earlier, i.e. abstract out emission of constructors to CGCXXABI rather than making the conditions more complex in CGM.<br><br>Can you please review this patch?<br></div></blockquote><div><br></div>LGTM.</div><div><br></div><div>John.</div></body></html>