This is the mail archive of the binutils@sources.redhat.com mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

turn on -relax by default for ia64


The ia64 abi specifies that the linker is supposed to add long branch
instructions as necessary for pc-relative calls that cannot reach their
destinations.  This was implemented some time ago, but since it uses
the relaxation hooks, one had to add -relax to the link line to get it
to work.

It's tiring to continue answering the same user questions in response
to the linker error one gets from the out-of-range branch, and somewhat 
silly that this isn't fixed up by default.

Addressed thus.


r~


        * emultempl/needrelax.em: New file.
        * emulparams/elf64_ia64.sh (EXTRA_EM_FILE): Reference it.
        * Makefile.am (eelf64_ia64.c): Depend on it.
        * Makefile.in: Rebuild.

Index: Makefile.am
===================================================================
RCS file: /cvs/cvsfiles/devo/ld/Makefile.am,v
retrieving revision 1.122
diff -c -p -d -r1.122 Makefile.am
*** Makefile.am	2001/08/11 18:30:57	1.122
--- Makefile.am	2001/08/13 20:21:46
*************** eelf64_aix.c: $(srcdir)/emulparams/elf64
*** 643,649 ****
    $(srcdir)/emultempl/elf32.em $(srcdir)/scripttempl/elf.sc ${GEN_DEPENDS}
  	${GENSCRIPTS} elf64_aix "$(tdir_elf64_aix)"
  eelf64_ia64.c: $(srcdir)/emulparams/elf64_ia64.sh \
!   $(srcdir)/emultempl/elf32.em $(srcdir)/scripttempl/elf.sc ${GEN_DEPENDS}
  	${GENSCRIPTS} elf64_ia64 "$(tdir_elf64_ia64)"
  eelf64_s390.c: $(srcdir)/emulparams/elf64_s390.sh \
    $(srcdir)/emultempl/elf32.em $(srcdir)/scripttempl/elf.sc ${GEN_DEPENDS}
--- 644,651 ----
    $(srcdir)/emultempl/elf32.em $(srcdir)/scripttempl/elf.sc ${GEN_DEPENDS}
  	${GENSCRIPTS} elf64_aix "$(tdir_elf64_aix)"
  eelf64_ia64.c: $(srcdir)/emulparams/elf64_ia64.sh \
!   $(srcdir)/emultempl/elf32.em $(srcdir)/emultempl/needrelax.em \
!   $(srcdir)/scripttempl/elf.sc ${GEN_DEPENDS}
  	${GENSCRIPTS} elf64_ia64 "$(tdir_elf64_ia64)"
  eelf64_s390.c: $(srcdir)/emulparams/elf64_s390.sh \
    $(srcdir)/emultempl/elf32.em $(srcdir)/scripttempl/elf.sc ${GEN_DEPENDS}
Index: emulparams/elf64_ia64.sh
===================================================================
RCS file: /cvs/cvsfiles/devo/ld/emulparams/elf64_ia64.sh,v
retrieving revision 1.7
diff -c -p -d -r1.7 elf64_ia64.sh
*** emulparams/elf64_ia64.sh	2001/06/16 15:23:09	1.7
--- emulparams/elf64_ia64.sh	2001/08/13 20:21:46
***************
*** 2,7 ****
--- 2,8 ----
  SCRIPT_NAME=elf
  ELFSIZE=64
  TEMPLATE_NAME=elf32
+ EXTRA_EM_FILE=needrelax
  OUTPUT_FORMAT="elf64-ia64-little"
  ARCH=ia64
  MACHINE=
Index: emultempl/needrelax.em
===================================================================
RCS file: needrelax.em
diff -N needrelax.em
*** /dev/null	Thu Aug 24 02:00:32 2000
--- emultempl/needrelax.em	Mon Aug 13 13:21:46 2001
***************
*** 0 ****
--- 1,40 ----
+ # This shell script emits a C file. -*- C -*-
+ #   Copyright (C) 2001 Free Software Foundation, Inc.
+ #
+ # This file is part of GLD, the Gnu Linker.
+ #
+ # This program is free software; you can redistribute it and/or modify
+ # it under the terms of the GNU General Public License as published by
+ # the Free Software Foundation; either version 2 of the License, or
+ # (at your option) any later version.
+ #
+ # This program is distributed in the hope that it will be useful,
+ # but WITHOUT ANY WARRANTY; without even the implied warranty of
+ # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ # GNU General Public License for more details.
+ #
+ # You should have received a copy of the GNU General Public License
+ # along with this program; if not, write to the Free Software
+ # Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
+ #
+ 
+ # This file is sourced from elf32.em.  It is used by targets for
+ # which relaxation is not just an optimization, but for correctness.
+ 
+ LDEMUL_BEFORE_ALLOCATION=need_relax_${EMULATION_NAME}_before_allocation
+ 
+ cat >>e${EMULATION_NAME}.c <<EOF
+ 
+ static void need_relax_${EMULATION_NAME}_before_allocation PARAMS ((void));
+ 
+ static void
+ need_relax_${EMULATION_NAME}_before_allocation ()
+ {
+   /* Call main function; we're just extending it.  */
+   gld${EMULATION_NAME}_before_allocation ();
+ 
+   /* Force -relax on if not doing a relocatable link.  */
+   if (! link_info.relocateable)
+     command_line.relax = true;
+ }
+ EOF


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]