diff options
| author | Khem Raj <kraj@mvista.com> | 2006-11-21 02:50:42 +0000 |
|---|---|---|
| committer | Khem Raj <kraj@mvista.com> | 2006-11-21 02:50:42 +0000 |
| commit | f032ccaa14a3c7bef2fc7ac2e24fd4613fa95428 (patch) | |
| tree | 5c2f30986d6e3d757c45d57c57735b2363bf66ab | |
| parent | 4dd72a937aed8056163addafe74fc38fa906db3b (diff) | |
Use constant pool instead of and adr
instruction in unoptimized builds and thumb: thanks Paul Brook
| -rw-r--r-- | ldso/ldso/arm/dl-sysdep.h | 26 |
1 files changed, 11 insertions, 15 deletions
diff --git a/ldso/ldso/arm/dl-sysdep.h b/ldso/ldso/arm/dl-sysdep.h index c8e1b9851..c5ec891df 100644 --- a/ldso/ldso/arm/dl-sysdep.h +++ b/ldso/ldso/arm/dl-sysdep.h @@ -110,24 +110,20 @@ elf_machine_load_address (void) extern void __dl_start asm ("_dl_start"); Elf32_Addr got_addr = (Elf32_Addr) &__dl_start; Elf32_Addr pcrel_addr; -#if !defined __thumb__ +#if defined __OPTIMIZE__ && !defined __thumb__ asm ("adr %0, _dl_start" : "=r" (pcrel_addr)); #else + /* A simple adr does not work in Thumb mode because the offset is + negative, and for debug builds may be too large. */ int tmp; - /* The above adr will not work on thumb because it - * is negative. The only safe way is to temporarily - * swap to arm. - */ - asm( ".align 2\n" - " bx pc\n" - " nop \n" - " .arm \n" - " adr %0, _dl_start\n" - " .align 2\n" - " orr %1, pc, #1\n" - " bx %1\n" - " .force_thumb\n" - : "=r" (pcrel_addr), "=&r" (tmp)); + asm ("adr %1, 1f\n\t" + "ldr %0, [%1]\n\t" + "add %0, %0, %1\n\t" + "b 2f\n\t" + ".align 2\n\t" + "1: .word _dl_start - 1b\n\t" + "2:" + : "=r" (pcrel_addr), "=r" (tmp)); #endif return pcrel_addr - got_addr; } |