From 60a019a089177f3f58f636580857414d769b8c11 Mon Sep 17 00:00:00 2001 From: Thomas Preud'homme Date: Tue, 14 Jun 2016 12:02:53 +0100 Subject: [PATCH] Fix feature checks based on ARM architecture value 2016-06-14 Thomas Preud'homme bfd/ * elf32-arm.c (using_thumb_only): Force review of arch check logic for new architecture. (using_thumb2): Try Tag_THUMB_ISA_use first and check for exact arch value then. Force review of arch check logic for new architecture. (arch_has_arm_nop): Update and fix arch check logic. Force review of that logic for new architecture. (arch_has_thumb2_nop): Remove. (elf32_arm_tls_relax): Use using_thumb2 instead of above function. (elf32_arm_final_link_relocate): Likewise but using thumb2. --- bfd/ChangeLog | 13 +++++++++++++ bfd/elf32-arm.c | 52 +++++++++++++++++++++++++++++++++---------------- 2 files changed, 48 insertions(+), 17 deletions(-) diff --git a/bfd/ChangeLog b/bfd/ChangeLog index 53c7c8848d1..7e64f86ec08 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,16 @@ +2016-06-14 Thomas Preud'homme + + * elf32-arm.c (using_thumb_only): Force review of arch check logic for + new architecture. + (using_thumb2): Try Tag_THUMB_ISA_use first and check + for exact arch value then. Force review of arch check logic for new + architecture. + (arch_has_arm_nop): Update and fix arch check logic. Force review of + that logic for new architecture. + (arch_has_thumb2_nop): Remove. + (elf32_arm_tls_relax): Use using_thumb2 instead of above function. + (elf32_arm_final_link_relocate): Likewise but using thumb2. + 2016-06-14 Alan Modra * bfd-in.h (bfd_my_archive): Delete. diff --git a/bfd/elf32-arm.c b/bfd/elf32-arm.c index 20a19891242..8de01b46671 100644 --- a/bfd/elf32-arm.c +++ b/bfd/elf32-arm.c @@ -3520,6 +3520,11 @@ using_thumb_only (struct elf32_arm_link_hash_table *globals) arch = bfd_elf_get_obj_attr_int (globals->obfd, OBJ_ATTR_PROC, Tag_CPU_arch); + /* Force return logic to be reviewed for each new architecture. */ + BFD_ASSERT (arch <= TAG_CPU_ARCH_V8 + || arch == TAG_CPU_ARCH_V8M_BASE + || arch == TAG_CPU_ARCH_V8M_MAIN); + if (arch == TAG_CPU_ARCH_V6_M || arch == TAG_CPU_ARCH_V6S_M || arch == TAG_CPU_ARCH_V7E_M @@ -3535,9 +3540,25 @@ using_thumb_only (struct elf32_arm_link_hash_table *globals) static bfd_boolean using_thumb2 (struct elf32_arm_link_hash_table *globals) { - int arch = bfd_elf_get_obj_attr_int (globals->obfd, OBJ_ATTR_PROC, - Tag_CPU_arch); - return arch == TAG_CPU_ARCH_V6T2 || arch >= TAG_CPU_ARCH_V7; + int arch; + int thumb_isa = bfd_elf_get_obj_attr_int (globals->obfd, OBJ_ATTR_PROC, + Tag_THUMB_ISA_use); + + if (thumb_isa) + return thumb_isa == 2; + + arch = bfd_elf_get_obj_attr_int (globals->obfd, OBJ_ATTR_PROC, Tag_CPU_arch); + + /* Force return logic to be reviewed for each new architecture. */ + BFD_ASSERT (arch <= TAG_CPU_ARCH_V8 + || arch == TAG_CPU_ARCH_V8M_BASE + || arch == TAG_CPU_ARCH_V8M_MAIN); + + return (arch == TAG_CPU_ARCH_V6T2 + || arch == TAG_CPU_ARCH_V7 + || arch == TAG_CPU_ARCH_V7E_M + || arch == TAG_CPU_ARCH_V8 + || arch == TAG_CPU_ARCH_V8M_MAIN); } /* Create .plt, .rel(a).plt, .got, .got.plt, .rel(a).got, .dynbss, and @@ -3742,19 +3763,16 @@ arch_has_arm_nop (struct elf32_arm_link_hash_table *globals) { const int arch = bfd_elf_get_obj_attr_int (globals->obfd, OBJ_ATTR_PROC, Tag_CPU_arch); - return arch == TAG_CPU_ARCH_V6T2 - || arch == TAG_CPU_ARCH_V6K - || arch == TAG_CPU_ARCH_V7 - || arch == TAG_CPU_ARCH_V7E_M; -} -static bfd_boolean -arch_has_thumb2_nop (struct elf32_arm_link_hash_table *globals) -{ - const int arch = bfd_elf_get_obj_attr_int (globals->obfd, OBJ_ATTR_PROC, - Tag_CPU_arch); - return (arch == TAG_CPU_ARCH_V6T2 || arch == TAG_CPU_ARCH_V7 - || arch == TAG_CPU_ARCH_V7E_M); + /* Force return logic to be reviewed for each new architecture. */ + BFD_ASSERT (arch <= TAG_CPU_ARCH_V8 + || arch == TAG_CPU_ARCH_V8M_BASE + || arch == TAG_CPU_ARCH_V8M_MAIN); + + return (arch == TAG_CPU_ARCH_V6T2 + || arch == TAG_CPU_ARCH_V6K + || arch == TAG_CPU_ARCH_V7 + || arch == TAG_CPU_ARCH_V8); } static bfd_boolean @@ -8987,7 +9005,7 @@ elf32_arm_tls_relax (struct elf32_arm_link_hash_table *globals, if (!is_local) /* add r0,pc; ldr r0, [r0] */ insn = 0x44786800; - else if (arch_has_thumb2_nop (globals)) + else if (using_thumb2 (globals)) /* nop.w */ insn = 0xf3af8000; else @@ -9817,7 +9835,7 @@ elf32_arm_final_link_relocate (reloc_howto_type * howto, if (h && h->root.type == bfd_link_hash_undefweak && plt_offset == (bfd_vma) -1) { - if (arch_has_thumb2_nop (globals)) + if (thumb2) { bfd_put_16 (input_bfd, 0xf3af, hit_data); bfd_put_16 (input_bfd, 0x8000, hit_data + 2); -- 2.30.2