[PATCH] 3.3 Release fix on ARM - atomics

Renato Golin renato.golin at linaro.org
Mon May 13 05:56:46 PDT 2013


Identified a while before the release, there is an issue with 64-bit
atomics on ARM that was making Clang mis-compile a lot of code, including
Clang itself.

http://llvm.org/bugs/show_bug.cgi?id=15429

Attached is a patch proposed by Benjamin with the corrections to the test.

I'm not an expert on how Clang lowers C11 atomics, but the resulting IR
seems correct, and after self-hosting Clang with this patch, it managed to
pass all test-suite (as before), which is a good indication that the patch
does fix the problem.

Please review!

cheers,
--renato
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130513/96b2ef27/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: atomic_arm.patch
Type: application/octet-stream
Size: 6943 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130513/96b2ef27/attachment.obj>


More information about the cfe-commits mailing list