<div class="gmail_quote">On Fri, Mar 30, 2012 at 3:45 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="gmail_quote"><div class="im">On Fri, Mar 30, 2012 at 5:46 AM, David Chisnall <span dir="ltr"><<a href="mailto:csdavec@swan.ac.uk" target="_blank">csdavec@swan.ac.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

I can disable that if __LP64__ is not defined, but I think a better solution would be for clang to support __uint128_t in 32-bit mode...<br></blockquote><div><br></div></div><div>David, please revert this patch and discuss this on the mailing list until these issues are addressed. This is why we have pre-commit review.</div>

</div>
</blockquote></div><br><div>David has clarified that review did take place, but apparantly off-list. =/ I wish that weren't happening.</div><div><br></div><div>Anyways, could we at least disable this temporarily? Leaving the build broken isn't reasonable...</div>