[LLVMdev] [PATCH] 3.3 Release fix on ARM - atomics
renato.golin at linaro.org
Mon May 13 07:59:39 PDT 2013
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 =
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.
> 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!
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-dev