diff options
| author | Renato Golin <renato.golin@linaro.org> | 2014-07-22 20:59:41 +0000 |
|---|---|---|
| committer | Renato Golin <renato.golin@linaro.org> | 2014-07-22 20:59:41 +0000 |
| commit | 4a5db0a980021d6e44e736926af7f17c472ff299 (patch) | |
| tree | 889eaadc549a8e2657c3b2b5a0c81169246676f3 /lib/builtins/arm/sync-ops.h | |
| parent | ad18d609f6c6ef413f8930ce3fb7d91bf5d39d62 (diff) | |
| download | compiler-rt-4a5db0a980021d6e44e736926af7f17c472ff299.tar.gz | |
Fix incompatible assembly in ARM builtins library
Convert the CBNZ backward branch instruction to CMP and BNE
avoiding illegal backwards branch and making the assembly code
in synh-ops.h to be UAL compliant.
Patch by: Sumanth Gundapaneni
git-svn-id: https://llvm.org/svn/llvm-project/compiler-rt/trunk@213685 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/builtins/arm/sync-ops.h')
| -rw-r--r-- | lib/builtins/arm/sync-ops.h | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/lib/builtins/arm/sync-ops.h b/lib/builtins/arm/sync-ops.h index 87cc3c27c..5cca8f705 100644 --- a/lib/builtins/arm/sync-ops.h +++ b/lib/builtins/arm/sync-ops.h @@ -18,6 +18,7 @@ #define SYNC_OP_4(op) \ .p2align 2 ; \ .thumb ; \ + .syntax unified ; \ DEFINE_COMPILERRT_FUNCTION(__sync_fetch_and_ ## op) \ dmb ; \ mov r12, r0 ; \ @@ -25,13 +26,15 @@ ldrex r0, [r12] ; \ op(r2, r0, r1) ; \ strex r3, r2, [r12] ; \ - cbnz r3, LOCAL_LABEL(tryatomic_ ## op) ; \ + cmp r3, #0 ; \ + bne LOCAL_LABEL(tryatomic_ ## op) ; \ dmb ; \ bx lr #define SYNC_OP_8(op) \ .p2align 2 ; \ .thumb ; \ + .syntax unified ; \ DEFINE_COMPILERRT_FUNCTION(__sync_fetch_and_ ## op) \ push {r4, r5, r6, lr} ; \ dmb ; \ @@ -40,7 +43,8 @@ ldrexd r0, r1, [r12] ; \ op(r4, r5, r0, r1, r2, r3) ; \ strexd r6, r4, r5, [r12] ; \ - cbnz r6, LOCAL_LABEL(tryatomic_ ## op) ; \ + cmp r6, #0 ; \ + bne LOCAL_LABEL(tryatomic_ ## op) ; \ dmb ; \ pop {r4, r5, r6, pc} |