diff options
| -rw-r--r-- | clang/lib/Basic/Targets.cpp | 4 | ||||
| -rw-r--r-- | clang/test/CodeGen/x86_64-atomic-128.c | 29 |
2 files changed, 30 insertions, 3 deletions
diff --git a/clang/lib/Basic/Targets.cpp b/clang/lib/Basic/Targets.cpp index 7f5ee91d496..b0dc3356386 100644 --- a/clang/lib/Basic/Targets.cpp +++ b/clang/lib/Basic/Targets.cpp @@ -3292,10 +3292,8 @@ public: ComplexLongDoubleUsesFP2Ret = true; // x86-64 has atomics up to 16 bytes. - // FIXME: Once the backend is fixed, increase MaxAtomicInlineWidth to 128 - // on CPUs with cmpxchg16b MaxAtomicPromoteWidth = 128; - MaxAtomicInlineWidth = 64; + MaxAtomicInlineWidth = 128; } BuiltinVaListKind getBuiltinVaListKind() const override { return TargetInfo::X86_64ABIBuiltinVaList; diff --git a/clang/test/CodeGen/x86_64-atomic-128.c b/clang/test/CodeGen/x86_64-atomic-128.c new file mode 100644 index 00000000000..2069e455828 --- /dev/null +++ b/clang/test/CodeGen/x86_64-atomic-128.c @@ -0,0 +1,29 @@ +// RUN: %clang_cc1 -triple x86_64-linux-gnu -target-cpu core2 %s -S -emit-llvm -o - | FileCheck %s + +// All atomics up to 16 bytes should be emitted inline on x86_64. The +// backend can reform __sync_whatever calls if necessary (e.g. the CPU +// doesn't have cmpxchg16b). + +__int128 test_sync_call(__int128 *addr, __int128 val) { + // CHECK-LABEL: @test_sync_call + // CHECK: atomicrmw add i128 + return __sync_fetch_and_add(addr, val); +} + +__int128 test_c11_call(_Atomic __int128 *addr, __int128 val) { + // CHECK-LABEL: @test_c11_call + // CHECK: atomicrmw sub + return __c11_atomic_fetch_sub(addr, val, 0); +} + +__int128 test_atomic_call(__int128 *addr, __int128 val) { + // CHECK-LABEL: @test_atomic_call + // CHECK: atomicrmw or + return __atomic_fetch_or(addr, val, 0); +} + +__int128 test_expression(_Atomic __int128 *addr) { + // CHECK-LABEL: @test_expression + // CHECK: atomicrmw and + *addr &= 1; +} |
