[llvm-commits] [compiler-rt] r153735 - /compiler-rt/trunk/lib/atomic.c

Chandler Carruth chandlerc at google.com
Sat Mar 31 02:20:24 PDT 2012


On Fri, Mar 30, 2012 at 3:45 PM, Chandler Carruth <chandlerc at google.com>wrote:

> On Fri, Mar 30, 2012 at 5:46 AM, David Chisnall <csdavec at swan.ac.uk>wrote:
>
>> 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...
>>
>
> 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.
>

David has clarified that review did take place, but apparantly off-list. =/
I wish that weren't happening.

Anyways, could we at least disable this temporarily? Leaving the build
broken isn't reasonable...
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120331/8b12e266/attachment.html>


More information about the llvm-commits mailing list