[cfe-dev] [PATCH] 3.3 Release fix on ARM - atomics

Renato Golin renato.golin at linaro.org
Mon May 13 07:59:39 PDT 2013


Hi Rafael,

As you mentioned in the bug, we should only apply this change when
hard-float is set, which it is by default on armv7a, I presume.

>From that part of the code, I can infer that by the time
"MaxAtomicPromoteWidth = 64;", the variable SoftFloat is not properly set,
so a simple "if (!SoftFloat)" won't cut in there. It seems SoftFloat is
being set on HandleTargetFeatures() which is a virtual method, probably
called indirectly. Do you have a better place to set MaxAtomicInlineWidth =
64?

cheers,
--renato


On 13 May 2013 13:56, Renato Golin <renato.golin at linaro.org> wrote:

> 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-dev/attachments/20130513/c2b8fbc4/attachment.html>


More information about the cfe-dev mailing list