Re: commit eae0b5c3b2d6b
authorAlan Modra <amodra@gmail.com>
Thu, 27 Aug 2020 12:24:16 +0000 (21:54 +0930)
committerAlan Modra <amodra@gmail.com>
Thu, 27 Aug 2020 12:26:33 +0000 (21:56 +0930)
PR 26416
* elf64-alpha.c (elf64_alpha_relax_tls_get_addr): Correct "dynamic".

bfd/ChangeLog
bfd/elf64-alpha.c

index 646036483a9cc489bd89cfb1ffcbced19e007d5d..fb998c24986bab251c28d4c784d94e276bf339ed 100644 (file)
@@ -1,3 +1,8 @@
+2020-08-27  Alan Modra  <amodra@gmail.com>
+
+       PR 26416
+       * elf64-alpha.c (elf64_alpha_relax_tls_get_addr): Correct "dynamic".
+
 2020-08-26  Nick Clifton  <nickc@redhat.com>
 
        PR 26433
index fd1f4b20be138fc878d7784f71edf0a3741fb546..7b708f1e9d91b9893d30dfdd38cbc16e6663bc87 100644 (file)
@@ -3464,9 +3464,8 @@ elf64_alpha_relax_tls_get_addr (struct alpha_relax_info *info, bfd_vma symval,
   bfd_boolean dynamic, use_gottprel;
   unsigned long new_symndx;
 
-  if (info->h == NULL)
-    return TRUE; /* FIXME: Should this be return FALSE ?  */
-  dynamic = alpha_elf_dynamic_symbol_p (&info->h->root, info->link_info);
+  dynamic = (info->h != NULL
+            && alpha_elf_dynamic_symbol_p (&info->h->root, info->link_info));
 
   /* If a TLS symbol is accessed using IE at least once, there is no point
      to use dynamic model for it.  */