From f69f7529bc8b561ddc32a5057da6d74d90dd9c94 Mon Sep 17 00:00:00 2001 From: Khem Raj Date: Sat, 12 Jun 2021 08:42:36 -0700 Subject: [PATCH] Do not force thumb mode directive .thumb_func was not switching mode until [1] so it did not show up but now that .thumb_func (without argument) is switching mode, its causing build failures on armv6 ( rpi0 ) even when build is explicitly asking for this file to be built with -marm (ARM mode), therefore use DEFINE_COMPILERRT_FUNCTION macro to add function header which considers arch and mode from compiler cmdline to decide if the function is built using thumb mode or arm mode. [1] https://reviews.llvm.org/D101975 Signed-off-by: Khem Raj --- compiler-rt/lib/builtins/arm/sync-ops.h | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/compiler-rt/lib/builtins/arm/sync-ops.h b/compiler-rt/lib/builtins/arm/sync-ops.h index 7a26170741ad..d914f9d3a109 100644 --- a/compiler-rt/lib/builtins/arm/sync-ops.h +++ b/compiler-rt/lib/builtins/arm/sync-ops.h @@ -16,9 +16,8 @@ #define SYNC_OP_4(op) \ .p2align 2; \ - .thumb; \ .syntax unified; \ - DEFINE_COMPILERRT_THUMB_FUNCTION(__sync_fetch_and_##op) \ + DEFINE_COMPILERRT_FUNCTION(__sync_fetch_and_##op) \ DMB; \ mov r12, r0; \ LOCAL_LABEL(tryatomic_##op) : ldrex r0, [r12]; \ @@ -31,9 +30,8 @@ #define SYNC_OP_8(op) \ .p2align 2; \ - .thumb; \ .syntax unified; \ - DEFINE_COMPILERRT_THUMB_FUNCTION(__sync_fetch_and_##op) \ + DEFINE_COMPILERRT_FUNCTION(__sync_fetch_and_##op) \ push {r4, r5, r6, lr}; \ DMB; \ mov r12, r0; \