Code review comment for lp:~linaro-toolchain-dev/gcc-linaro/64bitatomics

Revision history for this message
Dr. David Alan Gilbert (davidgil-uk) wrote :

On 17 October 2011 09:23, Ramana Radhakrishnan
<email address hidden> wrote:
>
>> +UNSUPPORTED: gcc.dg/di-longlong64-sync-1.c
>> +UNSUPPORTED: gcc.dg/di-sync-multithread.c
>
> Why is this unsupported on x86-64 ?

They're enabled by a check that the target has the suitable instruction
available at runtime; this is check_effective_target_sync_longlong in
target-supports.exp; for
ARM I do an ldrexd to check it's got it; I didn't look to add an
appropriate test
for other arch's, but the placeholder is there so they can.

(Another way would be for arch's where long is 64bit it would be possible
to pass the buck to check_effective_target_sync_int_long).

Dave

« Back to merge proposal